[GitHub] jmeter pull request #360: Reordered menus. With some drive-by tidying up.

[ham1]

GitHub user ham1 opened a pull request:

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

    Reordered menus. With some drive-by tidying up.

    ## Description
    
    Added ability for MenuFactory to add separators to menus.
    Some TestBeanGUI refactoring for reformatting.
    New Spock test.
    Refactored AbstractThreadGroupGui menu creation.
    Cleaned up horrible if sequence and unneeded variables
    
    ## How Has This Been Tested?
    `ant test` and manually
    
    ## Screenshots (if appropriate):
    
    
![test-plan_menu](https://user-images.githubusercontent.com/3393038/33964457-6dbbb87c-e050-11e7-816c-c409da44fcc5.png)
    
![thread](https://user-images.githubusercontent.com/3393038/33964458-6ddc6f4a-e050-11e7-9f49-06168c5ba20b.png)
    
![sampler](https://user-images.githubusercontent.com/3393038/33964456-6d943dce-e050-11e7-9e10-6b6cfe200fbc.png)
    
    ## Types of changes
    - Bug fix (non-breaking change which fixes an issue)
    
    ## Checklist:
    <!--- Go over all the following points, and put an `x` in all the boxes 
that apply. -->
    <!--- If you're unsure about any of these, don't hesitate to ask. We're 
here to help! -->
    - [x] My code follows the [code style][style-guide] of this project.
    - [x] I have updated the documentation accordingly.
    
    [style-guide]: https://wiki.apache.org/jmeter/CodeStyleGuidelines


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ham1/jmeter menu_sort_order

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/jmeter/pull/360.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #360
    
----
commit fc7e50fd28cde7760d0bc581f3513823abb0b2f3
Author: Graham Russell <gra...@ham1.co.uk>
Date:   2017-12-13T21:43:51Z

    Reordered menus. With some drive-by tidying up.
    Added ability for MenuFactory to add separators to menus.
    Some TestBeanGUI refactoring for reformatting.
    New Spock test.
    Refactored AbstractThreadGroupGui menu creation.
    Cleaned up horrible if sequence and unneeded variables

----


---