Bug 12510 - Need Home menu icon for advance settings and other misc.
: Need Home menu icon for advance settings and other misc.
Status: CLOSED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: UI Skin
: unspecified
: PC Windows Vista
: P1 major (vote)
: 7.4.0
Assigned To: Wadzinski Tom
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-06-23 17:28 UTC by ndijulio
Modified: 2009-10-05 14:25 UTC (History)
5 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description ndijulio 2009-06-23 17:28:52 UTC
Couple options:

1. Create a generic "Shortcut" icon for the home menu when misc. items from advanced settings are included on the Home menu.  This proposal will scale as the setting options will no doubt expand and contract.

2. Reuse the advance settings icon (currently being designed) for any item that is added to Home menu.
Comment 1 ndijulio 2009-06-23 17:29:23 UTC
This icon would be used on Baby, Jive, and Fab4...
Comment 2 Ben Klaas 2009-06-23 19:42:45 UTC
a suggestion-- any settings item, including anything in advanced, that does not have an icon designated for it gets the generic "wrench" settings icon.
Comment 3 Weldon Matt 2009-07-13 22:03:54 UTC
Noah, what's your opinion on Ben's suggestion?  FYI, it's being retargeted for CXR.
Comment 4 ndijulio 2009-07-14 10:48:32 UTC
I think it is consistent with the way we are treating multiple search icons.  To be clear, these default icons would only be used if a user added them to the home menu.
Comment 5 ndijulio 2009-07-15 09:47:49 UTC
We now have both a "Shortcut" icon as well as a method for advanced settings to use the generic "Settings" icon when added to the home menu.  In all cases, a supporting text string will be either to the right of the icon or directly below.

Matt-Please close if you give this approach the green light.
Comment 6 Ben Klaas 2009-07-16 06:58:59 UTC
cc:ing Tom, this is the bug for the "fallbackIconStyle" that we talked about.

The idea is that home menu items would have a style (which is often dynamically derived from its item id value) as well as a fallbackStyle, which would take effect if the style was not defined in the skin. This would avoid a situation where an item was promoted from advanced to home and it has no icon.

Not going to happen for CAT, but I think it's major not minor
Comment 7 Ben Klaas 2009-08-03 08:02:15 UTC
Tom, assigning this one your way. We need some C support for the fallback style and I think we'll be able to put this bug to rest...
Comment 8 Ben Klaas 2009-08-03 08:03:09 UTC
*** Bug 12512 has been marked as a duplicate of this bug. ***
Comment 9 Ben Klaas 2009-08-19 14:24:27 UTC
*** Bug 13491 has been marked as a duplicate of this bug. ***
Comment 10 Wadzinski Tom 2009-09-09 15:54:24 UTC
Richard suggested a possible easy fix: use the parent icon, when adding the item to the home menu.
Comment 11 ndijulio 2009-09-09 16:01:03 UTC
Using a parent icon when sub menus do not contain developed icons makes a lot of sense.  We will need to look at this on a case by case basis, no?  Part of the strength of a "shortcut" icon is that it is by definition generic.  If we can be smarter about the icon defaults then I am all for it...
Comment 12 SVN Bot 2009-09-09 20:02:42 UTC
 == Auto-comment from SVN commit #7484 to the jive repo by tom ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7484 ==

Fixed Bug: 12510 +1
- Use advanced settings icon for any item that doesn't explictly define a iconStyle
Comment 13 ndijulio 2009-09-10 10:07:39 UTC
What is the proposed default for new features, new apps, etc.?
Comment 14 James Richardson 2009-10-05 14:25:46 UTC
This bug has been marked as fixed in the 7.4.0 release version of SqueezeBox Server!
    * SqueezeCenter: 28672
    * Squeezebox 2 and 3: 130
    * Transporter: 80
    * Receiver: 65
    * Boom: 50
    * Controller: 7790
    * Radio: 7790  

Please see the Release Notes for all the details: http://wiki.slimdevices.com/index.php/Release_Notes

If you haven't already, please download and install the new version from http://www.logitechsqueezebox.com/support/download-squeezebox-server.html

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.