Bug 56228

Summary: View Results Tree : Improve ergonomy by changing placement of Renderers and allowing custom ordering
Product: JMeter Reporter: Philippe Mouawad <p.mouawad>
Component: MainAssignee: JMeter issues mailing list <issues>
Severity: enhancement CC: p.mouawad
Priority: P2    
Version: 2.11   
Target Milestone: ---   
Hardware: Other   
OS: All   
Attachments: Proposal
Patch proposal

Description Philippe Mouawad 2014-03-06 22:35:44 UTC
As per mailing list discussion:

Discussion excerpt:
Philippe M. proposal
    *View Results Tree:*
    - I feel  the select box at the bottom left of component allowing selection
    of a Viewer is misplaced. I often switch between Viewers during scripting
    and I think it should a Tabbed Pane on the Top Center Right Part of View
    Results Tree or something better if you have ideas.  This would make it
    more visible and allow easy switching. Currently you lose a lot of time
    switching and finding the right component
    I also think we should at least change their display priority, I tend to
    think it should be something like
    - Text
    - Regexp Tester
    - CSS / JQUERY Test
    - XPATH Tester
    - HTML
    - HTML (download) => Very annoying that it blocks screen to download
    - Document
    - JSON => Not very useful currently
    - XML  => Do you use it ?

Milamber note:
Yes it's a good idea to change order. Except perhaps Text plain view, the order for others renderers can be different in function of situation.
Perhaps, introduce a property for ordering the renderers?
Comment 1 Philippe Mouawad 2014-03-06 22:41:25 UTC
Created attachment 31373 [details]

New property introduced which controls order:

Note renderers not mentionned in list will be added after those mentionned to handle correctly plugins or errors.
Comment 2 Philippe Mouawad 2014-03-06 22:57:55 UTC
Created attachment 31374 [details]
Patch proposal
Comment 3 Sebb 2014-03-07 02:01:35 UTC
(In reply to Philippe Mouawad from comment #1)

> com.ubikingenierie.loadpack.json.render.RenderJsonAsXML,com.
> ubikingenierie.loadpack.json.render.RenderAsJsonRenderer,com.ubikingenierie.
> loadpack.json.render.RenderJsonAsXML,com.ubikingenierie.loadpack.json.render.
> RenderAsJsonRenderer

Why are these classes not in the JMeter package structure?
Comment 4 Philippe Mouawad 2014-03-07 14:45:53 UTC
These are plugins, forget about them, they will not be included.

Otherwise shall I commit this ?
Comment 5 Sebb 2014-03-07 15:15:58 UTC
The list is quite awkward to read.

I suggest allowing a shorthand - instead of 


one could use


i.e. if the class name starts with "." (which is not valid) then prepend

Comment 6 Philippe Mouawad 2014-03-07 17:27:54 UTC
Ok, I will implement and commit.
Comment 7 Philippe Mouawad 2014-03-10 22:37:13 UTC
Date: Mon Mar 10 22:34:31 2014
New Revision: 1576110

URL: http://svn.apache.org/r1576110
Bug 56228 - View Results Tree : Improve ergonomy by changing placement of Renderers and allowing custom ordering
Bugzilla Id: 56228