Bugzilla – Bug 12755
missing icons in settings menu
Last modified: 2009-10-06 09:21:56 UTC
Created attachment 5437 [details] missing icons in settings menu in the top level settings menu, I see two items that do not have icons- "Name" - this was promoted from Advanced Settings to Settings with the recent (last night) reorg that Tom did per Weldon's org spec. "Select Skin" - I'm not 100% sure this is going to appear on device, but it does appear on desktop. My recommendation is to move it into the Screen submenu, as it deals with how things are rendered on the screen. If we do that, we don't need an icon. Weldon, what say you?
FYI the entire icon library will have a large update this Friday, including the remaining setting icons with the latest org. spec. Let me know if we need to add Select Skin ASAP.
Yeah... I have a feeling, with the constant minor controversies around font sizes etc, that users are going to need some kind of "text size" setting (probably eventually supported with the "size" button on the classic remote). I don't think that needs to literally translate into the "skin" per se, but I held off bugging you about this until the skins were completed.
Weldon, what do you think about moving Select Skin into the Screen submenu?
FYI the "text size" setting is already listed in the wiki as part of the settings menu redesign: http://embargo.wiki.slimdevices.com/index.php/Fab4_Settings Those settings are what we should be designing for re: icons.
All the icons seem to be present now as of r6537 (with the latest SC), except for "Squeezebox Name." Noah, do we have an asset for this? FYI this is targeted for CXR...
Matt- "Squeezebox Name" = Player Name correct? What is the final nomenclature? There is an icon for changing the player name in the directory, however, it is labeled > icon_settings_name.png Ben-please pull in this asset if not done so already. FYI we did not create a "Select Skin" icon. Please let me know if we need to do so.
We don't need a "select skin" icon. If this feature survives it will be a submenu under "screen settings." (I would like to see if we can make this one-and-the-same with "text size," but in terms of the icon it's moot - either way, no icon needed)
Ben- reassigning to you. Please pull in the updated assets.
fixed in r6594
Weldon- on comment #7, it's probably never going to be synonymous with text size, as desktop squeezeplay will have the ability of loading skins of different resolutions (i.e., baby, controller, fab4, and *gasp* something actually optimized for the desktop). But I agree it should go in screen settings, and I've already placed it there. Ergo, no icon issue in the settings menu with that one now.
This bug has been fixed in the latest release of MySqueezebox.com (formally known as SqueezeNetwork)! If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.