pippin;564128 Wrote: 
> Isn't this a completely client-side thing?
> 
Yes I'm pretty sure it's a client side thing when looking at the code,
I also can't find any server related commits regarding this. The change
was introduced in SqueezePlay r8987, so that explains why Jim didn't see
it in the latest nightly yet.

After trying it out a bit I think the news compared to the old
"Settings/Home Menu" is:
- You can reorder menus

This is a great enhancement, I'm just still wondering if context menus
is the best way to do it.

As far as I can see I can do the other changes on my Radio with 7.5
firmware through the "Settings/Home Menu", but I might be missing
something obvious. The main difference is of course that you use
context menus to do menu configuration instead of the "Settings/Home
Menu" alternative we had before.

I just built the latest 7.6 SqueezePlay and tried it out a bit on a
laptop and some things I noticed:
- It's not possible to hide the "Settings" menu, as long as this isn't
possible I think the Restore option should be somewhere inside the
Settings menu. In fact, there already is a "Settings/Home Menu/Restore
Defaults" that gets you back to the default setup where everything is
visible. Adding the "Restore Home Menu Items" option under
"Settings/Home Menu" would make more sense to me than cluttering the
Home menu with it. 
- "Settings/Home Menu" now only contains "Restore Defaults", so there
is no longer any way to hide/show menus there, context menus are the
only way to do this.
- The "Move up/Move down" options in the context menu closes it, so if
you want to move and item several steps it means you will need to
launch the context menu for each step you like to move it.
- The "Restore Home Menu Items" option have sub menus for each item, so
you can just restore a single item. This is good and different from the
"Settings/Home Menu/Restore Defaults" which restores everything and
don't makes it possible to just restore a single menu item.
- It's possible to both move a menu one step and to move it to
top/bottom
- When a menu is promoted to the home menu it will still be visible
also inside the sub menu, this is probably the expected behavior I'm
just mention it in case someone had a question about that. 
- The code for the context menu is hard coded for these specific items,
I can't find any possibility for third parties to add more items without
patching the source code. I'm not sure that's a big issue, but I would
have preferred if it had been implemented with that possibility.
However, this is pretty similar to other code in SqueezePlay which also
misses some third party API hooks that would be good to have.
- Since this is a client side context menu, it means that it will be
harder to get a generic server generated context menu on all menu items
where third parties can add items, similar to how the third party
Context Menu plugin works on IP3K devices. Although, I suppose
something similar could still be implemented on the client side.


After trying it out a bit, I have to say that I stick with my initial
comments:
- It was easier to hide/promote several menus with the old solution
where you just checked/unchecked some checkboxes under "Settings/Home
Menu"
- Re-ordering would be a lot easier if it was possible to move a menu
several steps without re-launching the context menu for each step
- IMHO, menu configuration should be put away somewhere under settings
since this is something you don't change during normal usage,
especially as it's not possible to hide the Settings anyway. If it was
possible to hide the "Settings" menu the situation would be different
because then we would need some way to show hidden menus again. Another
reason for having it under Settings is that I can see a possible future
feature that makes it possible to lock the device from
re-configuration, for example during parties or in kids rooms. To make
this easier, it would be better if all configuration was done under the
Settings menu.

Generally, I also think it's important to think about the use case:
- Who is going to do these changes ?
- When and how often are they going to be done for a specific device ?
- Are you typically going to reconfigure a single menu or several menus
at the same time ?

Configuration options you don't access very often should IMHO be put
inside "Settings" menu and not clutter the rest of the interface where
they might confuse normal users who just want to listen to the music.


-- 
erland

Erland Isaksson ('My homepage' (http://erland.isaksson.info))
(Developer of 'many plugins/applets'
(http://wiki.slimdevices.com/index.php/User:Erland). If my answer
helped you and you like to encourage future presence on this forum
and/or third party plugin/applet development, 'donations are always
appreciated' (http://erland.isaksson.info/donate))
------------------------------------------------------------------------
erland's Profile: http://forums.slimdevices.com/member.php?userid=3124
View this thread: http://forums.slimdevices.com/showthread.php?t=80658

_______________________________________________
beta mailing list
[email protected]
http://lists.slimdevices.com/mailman/listinfo/beta

Reply via email to