Bug 5241 - Cosmetic issues with song info screen
: Cosmetic issues with song info screen
Status: CLOSED FIXED
Product: SB Controller
Classification: Unclassified
Component: Browser
: unspecified
: Macintosh Other
: P1 normal (vote)
: 7.0
Assigned To: ndijulio
:
Depends on: 5554
Blocks:
  Show dependency treegraph
 
Reported: 2007-08-12 21:37 UTC by Blackketter Dean
Modified: 2008-05-15 12:59 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
rough idea of the full screen artwork (108.62 KB, image/png)
2007-09-25 17:12 UTC, Blackketter Dean
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Blackketter Dean 2007-08-12 21:37:14 UTC
1.  In the song info menu on Jive, when there is an item that can not be decended into (like File Format), no > should be displayed.

2.  There should be a space between the colon and the value of the items.

3.  Some items should be suppressed (Year: 0, Comment: 1

4.  Consider using a two-line format for each item, where the tag name and value are on separate lines.

5.  One item should be a full width cover art image, when available.
Comment 1 Richard Titmuss 2007-09-19 06:23:23 UTC
I think we need some artwork before this can be fixed, to be clear what we are trying to achieve.
Comment 2 Blackketter Dean 2007-09-19 08:39:13 UTC
noah: can you look at the song info screen and propose a new design?

Comment 3 Blackketter Dean 2007-09-21 12:00:37 UTC
The only artwork we need is the full-screen artwork info that will be one of the item choices from the info screen.

We can use the multi-line version that can handle small icons for this screen.  First line is the tag, the second (and third if wrapped) will be the value.    Display a > when you can go to the right and add Artwork as one of the items (with a small icon) that can be entered to noah's full screen design.
Comment 4 Ben Klaas 2007-09-25 15:34:32 UTC
notes on comment #1:

1. Fixed, SS r13272 Jive r570
2. Fixed, SS r13272
3. Workaround-- I couldn't get the full suppression to work, so a blank comment or year will result in the localized string NONE printing for the key value
4. I did a lot of experimenting with two line formatting in the player information setting, and came to the conclusion that it looks lousy and is not a net benefit.
5. didn't do anything on this item
Comment 5 Blackketter Dean 2007-09-25 17:10:04 UTC
Great.  Noah will be providing a full-screen cover art layout and graphics.  Then we can add a "Artwork" item for items that have cover art.  Going into that will show that full screen item.  It will look something like the attached, but wait for Noah before implementing.
Comment 6 Blackketter Dean 2007-09-25 17:12:16 UTC
Created attachment 2172 [details]
rough idea of the full screen artwork
Comment 7 Chris Owens 2007-10-30 09:25:45 UTC
Ping noah

Dean do you want this done for 7.0?
Comment 8 Blackketter Dean 2007-10-30 17:28:45 UTC
I'd like to get this basic screen done sooner rather than later.
Comment 9 Ben Klaas 2007-10-31 06:46:44 UTC
change 14246 has a fix for Comment#1 Item 3
Comment 10 Richard Titmuss 2007-11-04 16:01:57 UTC
Although Dean said he would like this done sooner rather than later it's not an MPQ item. Set the milestone to 7.1 and increased the to priority P1.
Comment 11 Blackketter Dean 2007-12-27 09:20:46 UTC
Richard: your comment here seems to be about the wrong bug.  I think that this is fixed.

Comment 12 James Richardson 2008-05-15 12:59:14 UTC
This bug has recently been fixed in the latest release of SqueezeCenter 7.0.1

Please try that version, if you still see the error, then reopen this bug.

To download this version, please navigate to: http://www.slimdevices.com/su_downloads.html