Bug 14966 - CM remote skins selected menu item asset incorrect
: CM remote skins selected menu item asset incorrect
Status: CLOSED FIXED
Product: SB Touch
Classification: Unclassified
Component: UI Skin
: unspecified
: All Other
: P1 normal (vote)
: 7.5.0
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-10-27 16:18 UTC by ndijulio
Modified: 2010-04-08 17:25 UTC (History)
2 users (show)

See Also:
Category: Task


Attachments
Asset - Context Menu selected item (3.67 KB, application/octet-stream)
2009-10-27 16:18 UTC, ndijulio
Details
Ref artwork (21.66 KB, image/png)
2009-10-27 16:31 UTC, ndijulio
Details
center asset for selected treatement tiles (30.30 KB, image/png)
2009-10-30 08:26 UTC, Ben Klaas
Details
Assets - context menu large text format button (72x448) (3.42 KB, application/octet-stream)
2009-12-02 09:31 UTC, ndijulio
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ndijulio 2009-10-27 16:18:37 UTC
Created attachment 6225 [details]
Asset - Context Menu selected item

See attached for correct assets.
Comment 1 ndijulio 2009-10-27 16:31:13 UTC
Created attachment 6227 [details]
Ref artwork
Comment 2 Ben Klaas 2009-10-30 08:23:12 UTC
Noah, is the ref artwork attached here correct for 10' UI? My two cents is that I think three items per screen is all we should do for 10' UI menus.

If we go with three items I believe I will need a new center tile asset for both selected and pressed because it doesn't work at that item height. I'll attach a screenshot.
Comment 3 Ben Klaas 2009-10-30 08:26:51 UTC
Created attachment 6243 [details]
center asset for selected treatement tiles

Because the center asset has a gradient, the behavior when it stretches to fill the 9-tile space is to tile. For these assets to work with a 3-item high CM, will need a new center asset that's tall enough to not tile like that.

Need this for selected and pressed. I think the other 8 tiles in each should all work as is.
Comment 4 SVN Bot 2009-10-30 08:56:08 UTC
 == Auto-comment from SVN commit #7981 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7981 ==

Bug: 14966
Description: skin fixes to make 10' UI CM not completely broken. Still need update for selected and pressed asset
Comment 5 ndijulio 2009-11-02 15:46:58 UTC
Ben, correct.  Much like the main menu screens *3* items is the most we can support visually in the large text mode.  Updated assets to follow.  Need to provide a "tall" version with an updated gradient.
Comment 6 ndijulio 2009-12-01 14:45:54 UTC
Ben- did you ever receive assets for the lrg view (10ft) CM buttons?
Comment 7 ndijulio 2009-12-02 09:31:23 UTC
Created attachment 6350 [details]
Assets - context menu large text format button (72x448)
Comment 8 ndijulio 2009-12-02 09:33:00 UTC
Reassigning over to you Ben.  The assets for the cm lrg text format skin are attached.  Let me know if you need anything further...
Comment 9 SVN Bot 2009-12-09 09:11:43 UTC
 == Auto-comment from SVN commit #8205 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8205 ==

Fixed Bug: 14966
Description: fix CM selected assets for remote skin
Comment 10 SVN Bot 2009-12-09 11:25:53 UTC
 == Auto-comment from SVN commit #8206 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8206 ==

Bug: 14966
Description: center remote skin CM vertically
Comment 11 Chris Owens 2010-04-08 17:25:39 UTC
This bug has been marked fixed in a released version of Squeezebox Server or the accompanying firmware or mysqueezebox.com release.

If you are still seeing this issue, please let us know!