Bug 14122 - Need placeholder to fill artwork space while loading artwork
: Need placeholder to fill artwork space while loading artwork
Status: CLOSED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: UI
: unspecified
: Other Other
: P3 normal (vote)
: 7.5.x
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-09-18 09:50 UTC by Andy Grundman
Modified: 2010-09-13 12:01 UTC (History)
6 users (show)

See Also:
Category: Feature


Attachments
server-side patch to sent home_menu window style for menus that contain icons (607 bytes, patch)
2010-06-03 12:40 UTC, Ben Klaas
Details | Diff
client-side patch to tweak misc skins to style correctly for expanded home_menu style with loading icons (1.96 KB, patch)
2010-06-03 12:42 UTC, Ben Klaas
Details | Diff
updated client-side patch with small fix for WQVGA skin (1.88 KB, patch)
2010-06-03 13:05 UTC, Ben Klaas
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andy Grundman 2009-09-18 09:50:45 UTC
The current way a menu containing artwork is rendered is a bit jarring, text is 
left-aligned and then the artwork appears and pushes the text to the right.  I 
think it would look nicer if the artwork area was populated by a placeholder 
item (something like an empty dashed box) while the art was being loaded, thus 
not causing the text to suddenly move.
Comment 1 Chris Owens 2009-09-21 10:17:02 UTC
This apparently works for local content and the home menu, but not for remote icons.
Comment 2 Weldon Matt 2009-10-01 15:16:43 UTC
I'm confused about what's being described.  The behavior I've seen is that a partially-transparent rounded box appears to fill the gap, which is fine...
Comment 3 Andy Grundman 2009-10-01 15:31:58 UTC
Yeah we already have an icon, we just need to fix it so it's used for remote items i.e. Rhapsody albums.
Comment 4 Chris Owens 2010-03-15 18:05:09 UTC
7.4.x milestone is in the past
Comment 5 Ben Klaas 2010-06-03 12:40:51 UTC
Created attachment 6875 [details]
server-side patch to sent home_menu window style for menus that contain icons

the home_menu window style is used in squeezeplay for windows that contain icons on the left side of the menu but are otherwise styled like any plain text_list menu without icons. Currently that window style is used for the home menus only (hence the style name), but this can be extended to any XMLBrowse item that contains thumbnails.

there is a client-side part of this fix, but this patch does not cause UI breakage in its absence.
Comment 6 Ben Klaas 2010-06-03 12:42:15 UTC
Created attachment 6876 [details]
client-side patch to tweak misc skins to style correctly for expanded home_menu style with loading icons

this patch will allow for the "loading" icons that are used in the home menus to be extended to other XMLBrowse content, e.g. Staff Picks, Internet Radio submenus, napster albums, etc.
Comment 7 Ben Klaas 2010-06-03 13:05:46 UTC
Created attachment 6877 [details]
updated client-side patch with small fix for WQVGA skin
Comment 8 SVN Bot 2010-06-15 09:31:16 UTC
 == Auto-comment from SVN commit #30880 to the slim repo by bklaas ==
 == http://svn.slimdevices.com/slim?view=revision&revision=30880 ==

Bug: 14122
Description: if XMLBrowse feed menu items have images (thumbnails) in them, use window style 'home_menu'
home_menu window style is the same as text_list but puts in a placeholder icon for items with no thumbnail
Comment 9 SVN Bot 2010-06-15 09:41:34 UTC
 == Auto-comment from SVN commit #30884 to the slim repo by bklaas ==
 == http://svn.slimdevices.com/slim?view=revision&revision=30884 ==

Bug: 14122
Description: fix bad if clause in previous checkin
Comment 10 SVN Bot 2010-06-15 09:50:33 UTC
 == Auto-comment from SVN commit #8858 to the jive repo by bklaas ==
 == http://svn.slimdevices.com/jive?view=revision&revision=8858 ==

Fixed Bug: 14122
along with server side checkin r30884 or later, XMLBrowse menus that have thumbnails for any/all items will now have the window style of 'home_menu', which will put a placeholder icon by default into place on the menu item before the thumbnail loads
Comment 11 Chris Owens 2010-08-04 15:49:51 UTC
Works, looks good.
Comment 12 Chris Owens 2010-09-13 11:33:05 UTC
This bug has now been fixed in a released version of the Squeezeplay firmware (SB Touch, SB Radio, and SB Controller).  If you are still seeing this bug, please make certain you are running firmware 9009 or higher, and re-open it.

Thanks!
Comment 13 Chris Owens 2010-09-13 12:01:46 UTC
Since these are fixed, they must not be patch_waiting after all