Allow multiple submenus (Issue #13 & Issue #26)

[thoro]

Moved the submenu creation into separate methods which allows for a
recursive menu generation and therefore multiple submenus

[fitztrev]

Could you please share a sample config file to demonstrate the nested menu functionality? I was unable to get it working.

[thoro]

I've added a sample section to the default config, I opted to use the Dictionary within Array syntax you used for the "hosts" property.

Maybe we could change the group definition into something similar to the item definition - to add additional properties!

[thoro]

I added a sample for the icon, I noticed that it won't expand tildes so I added that too.

[numberwhun]

I downloaded the newest version from the website (which I assume is the newest), and I am unable to get the sub-submenu's working.

Here is what I was hoping to support:

{
"hosts": [
{
"sublevel 1": [
{
"name": "test",
"cmd": ""
}
{
"sub-sublevel": [
{
"name": "test",
"cmd": ""
}
]
}
]
}
]
}

Is this possible?

[thoro]

Yes it is possible, just that the current master doesn't have this pull request embedded ;-)

And you are missing a comma:

"cmd": ""
}, <-----
{
"sub-sublevel": [

I've merged it in mine and you can download the code here: https://github.com/Thoro/shuttle

[numberwhun]

I typed that in and missed that, sorry! I just downloaded and tested and its working perfectly. Thank you! You guys rock! I will keep testing and open tickets if I find anything.

[fitztrev]

@thoro Just tested it. Works well. 👍

Would you have any desire to expand this same functionality to support submenus with .ssh/config files?

Host Group1/Subgroup1/web01.example.com

[thoro]

Yeah, but in that move I'd like to add subclasses for a Menu and a MenuItem, to better support future functionality. I'm having my eye on automation through applescript and a bunch of additional properties!

[thoro]

I added a bunch of functionality and prepared it for future extension via a class hierarchy for the menu items, I've also added "sort_alphabetical" as a option as requested in #47 - but maybe we have to rename my option (I think the name is not quite right)

[fitztrev]

Nested menus have now been added in 89e05b8. See a couple example config files to see how they can be implemented.

@thoro Thank you very much for your contribution.