Bug 65545 - Jmeter 5.4.1 vs 5.3 : content is no more auto fit into right panel. (see video)
Summary: Jmeter 5.4.1 vs 5.3 : content is no more auto fit into right panel. (see video)
Status: NEW
Alias: None
Product: JMeter
Classification: Unclassified
Component: Main (show other bugs)
Version: 5.4.1
Hardware: PC All
: P2 trivial (vote)
Target Milestone: JMETER_5.5
Assignee: JMeter issues mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-09-02 11:16 UTC by pierre.astruc
Modified: 2021-09-03 15:56 UTC (History)
0 users



Attachments
5.3 panel was better, it auto fit the content. (624.02 KB, video/mp4)
2021-09-02 11:16 UTC, pierre.astruc
Details
VRT UI issue (81.68 KB, application/x-zip-compressed)
2021-09-03 09:24 UTC, eR@SeR
Details
Fix ok for Synthesis Report, but VRT is now strange. (343.43 KB, video/mp4)
2021-09-03 12:42 UTC, pierre.astruc
Details

Note You need to log in before you can comment on or make changes to this bug.
Description pierre.astruc 2021-09-02 11:16:11 UTC
Created attachment 38008 [details]
5.3 panel was better, it auto fit the content.

Hello !

With Jmeter 5.3, the right panel was auto fitting the content of Synthesis Report.
With Jmeter 5.4.1, it's no more the case, the right panel is just sliding content out of the window.

Is there one option to keep the old behavior ?
In my opinion, it was better to see everything into the window (autofit).

Please find the attached video (Jmeter_autofit_panel_53_vs_54.mp4) to understand my request :)

Thanks !
Pierre
Comment 1 Felix Schumacher 2021-09-02 15:29:56 UTC
@Pierre, could you check next build from trunk or nightly and report back, whether this issue is fixed?

And I agree, it was nicer before, but didn't know how to fix it.

commit a7bdcb24a379ee804c200bfa3a119891c2875614
AuthorDate: Thu Sep 2 17:26:39 2021 +0200

    SplitPanel shows a scrollbar instead of shrinking the right panel in the main view
    
    This hopefully restores the way it was in JMeter 5.3 and before.
    
    Bugzilla Id: 65545
---
 .../src/main/java/org/apache/jmeter/gui/util/VerticalPanel.java     | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
Comment 2 Felix Schumacher 2021-09-02 16:05:41 UTC
commit 2e7c471734cf666b9e43f5613a740060c899a923
AuthorDate: Thu Sep 2 18:03:41 2021 +0200

    SplitPanel shows a scrollbar instead of shrinking the right panel in the main view
    
    Remove unused import left over from last commit and add missing changelog entry.
    
    Bugzilla Id: 65545
---
 src/core/src/main/java/org/apache/jmeter/gui/util/VerticalPanel.java | 1 -
 xdocs/changes.xml                                                    | 1 +
 2 files changed, 1 insertion(+), 1 deletion(-)
Comment 3 eR@SeR 2021-09-03 09:24:56 UTC
Created attachment 38010 [details]
VRT UI issue

Hello,

In the latest nightly, VRT UI doesn't look good. Tree results are hidden by default. When expanded, tree view vertical size is reduced. Please check the attachment.

Jmeter 5.5 2c511c6
Microsoft Windows 10 Enterprise 64-bit
java version "15.0.1" 2020-10-20
Comment 4 Felix Schumacher 2021-09-03 10:58:50 UTC
Sadly there seems to be more hidden here. My tests show, that the Tree Results View is extremely slow with the changes. I will revert the changes this evening and letting others more time to check for more problems that this change could introduce.
Comment 5 pierre.astruc 2021-09-03 12:42:32 UTC
Created attachment 38012 [details]
Fix ok for Synthesis Report, but VRT is now strange.
Comment 6 pierre.astruc 2021-09-03 12:43:15 UTC
Hi Felix,

Thanks for the quick fix ! It was a good fix for Synthesis Report in nightly #2c511c6. (see video)
Unfortunatly, I can also confirm that View Results Tree is now strange :p

Pierre
Comment 7 Felix Schumacher 2021-09-03 15:56:25 UTC
commit 093df18f7c9e5e68d5e617bf2a8f4179281d89a4
AuthorDate: Fri Sep 3 17:49:36 2021 +0200

    Revert "SplitPanel shows a scrollbar instead of shrinking the right panel in the main view"
    
    The reported problem was fixed with the patch (that is reverted here), but the Tree Results View
    was broken with it. So revert it for now.
    
    This reverts commit 2e7c471734cf666b9e43f5613a740060c899a923.
    This reverts commit a7bdcb24a379ee804c200bfa3a119891c2875614.
    
    Bugzilla Id: 65545
---
 .../src/main/java/org/apache/jmeter/gui/util/VerticalPanel.java    | 7 +++----
 xdocs/changes.xml                                                  | 1 -
 2 files changed, 3 insertions(+), 5 deletions(-)