Bug 64248 - View Results Tree : Different character sizes (All version of JMeter)
Summary: View Results Tree : Different character sizes (All version of JMeter)
Status: NEW
Alias: None
Product: JMeter
Classification: Unclassified
Component: Main (show other bugs)
Version: Nightly (Please specify date)
Hardware: PC All
: P2 normal (vote)
Target Milestone: JMETER_5.3.0
Assignee: JMeter issues mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-03-20 09:37 UTC by Markowski_JM
Modified: 2020-04-27 11:31 UTC (History)
1 user (show)


Attachments
Print screen of character sizes (28.54 KB, application/pdf)
2020-03-20 09:37 UTC, Markowski_JM 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Markowski_JM 2020-03-20 09:37:03 UTC 
Created attachment 37109 [details]
Print screen of  character sizes

Hi,

I send you the document with all the differences of character sizes and background in View Results Tree.

I use the version apache-jmeter-5.3-SNAPSHOT from 2020-03-19 with Darklaf solarized light

To standardize the size of the characters, it's possible to keep the large size among the 4 samples in document.

Thanks.
Comment 1 Vladimir Sitnikov 2020-03-20 12:35:18 UTC 
The issue there is
"Response data" uses org.apache.jmeter.visualizers.SearchTextExtension#SearchTextExtension while
"Request data" uses org.apache.jmeter.gui.util.JSyntaxSearchToolBar#JSyntaxSearchToolBar

In other words, it looks like code duplication with a slightly different implementation.


There's one more issue: search toolbar can't be zoomed because it uses a hard-coded font size.


I think we should factor FONT_SMALL and friends to UIDefautls, then we can consistently update it for LaF and zoom purposes.
Comment 2 Vladimir Sitnikov 2020-03-20 16:35:31 UTC 
I wonder if the search panel should appear only when ctrl+f / cmd+f is pressed.
Comment 3 Philippe Mouawad 2020-04-27 11:31:34 UTC 
@Vladimir,
Can you confirm this is fixed by:


https://github.com/apache/jmeter/pull/572

Thanks