Bug 12512 - Need to use "loading" icon for all iconlist templates with exception of album and playlists
: Need to use "loading" icon for all iconlist templates with exception of album...
Status: RESOLVED DUPLICATE of bug 12510
Product: SqueezePlay
Classification: Unclassified
Component: Browser
: unspecified
: All Other
: P1 major (vote)
: 7.4.0
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-06-23 17:49 UTC by ndijulio
Modified: 2009-09-08 09:28 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments
Ref artwork - loading icon (92.73 KB, image/png)
2009-06-23 17:49 UTC, ndijulio
Details
example of what happens when "loading" icon is used with unified text_list style (22.61 KB, image/png)
2009-06-23 20:51 UTC, Ben Klaas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ndijulio 2009-06-23 17:49:11 UTC
Created attachment 5357 [details]
Ref artwork - loading icon
Comment 1 Ben Klaas 2009-06-23 20:49:09 UTC
Richard, FYI this requires us to again split the text_list style, which is actually what we're now using for all menus regardless of whether they have art on the left or not, to icon_list and text_list window styles.
Comment 2 Ben Klaas 2009-06-23 20:51:29 UTC
Created attachment 5358 [details]
example of what happens when "loading" icon is used with unified text_list style

with a "loading" icon as a default in the 'icon' slot of menus, this forces a re-split of text_list and icon_list window styles again. Otherwise you end up with menus that look like this...
Comment 3 Ben Klaas 2009-06-25 08:12:20 UTC
partial fix in r6246. This will show a loading icon for items that just have not received their thumb icon from SC/SN (e.g. Pandora, Rhapsody, etc.)

For items that are added to the home menu with an iconStyle that doesn't exist in the skin (e.g., Select Skin, anything in advanced Settings), those items still show no icon. Don't have a solution for that yet.
Comment 4 Ben Klaas 2009-08-03 08:03:09 UTC
bug 12510 covers the remaining issue in this bug

*** This bug has been marked as a duplicate of bug 12510 ***