Bug 8748 - Review spacing in Library statistics->Total Playing Time
: Review spacing in Library statistics->Total Playing Time
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Player UI
: unspecified
: PC Windows XP
: -- normal (vote)
: ---
Assigned To: Adrian Smith
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-07-15 17:02 UTC by Mickey Gee
Modified: 2009-09-08 09:24 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mickey Gee 2008-07-15 17:02:23 UTC
In Settings->Information->Library statistics->Total Playing Time, the second line display shows "Total Playing Tim245:29:08". I suggest adding a little more space between the string and the numeric value.
Comment 1 Mickey Gee 2008-07-15 17:06:50 UTC
This is with text size set to Light Narrow and language of English.
Comment 2 Michael Herger 2008-07-16 00:54:34 UTC
Not much we can do except shorten the title string: the playing time is displayed right aligned. This "overlay" will overwrite whatever is written on the left, as its former use was things like the play note mainly.

Or we don't align the info to the right and have the lines scroll.
Comment 3 Chris Owens 2008-07-21 10:42:07 UTC
Dean says this should scroll instead of being truncated in the middle.  If necessary, the screensaver should be delayed until one loop of the scrolling has been completed.

cc'ing Brian since Dean thinks this would be a good policy.
Comment 4 Adrian Smith 2008-07-22 02:55:13 UTC
So this is going back on the decison made in bug 7865 - do we want all information screens to be left aligned and scroll in this case?

I don't mind changing it back, but it would be good to ensure we have conistency on this.  It does mean we have a problem with the signal strength which will need to be solved another way.

Have you concidered a narrower standard font for Boom - seems to me that we could shave something off the height and definately the width of the standard font?
Comment 5 Michael Herger 2008-07-22 04:05:16 UTC
It's always a good idea to include Dean in such discussions.
Comment 6 Blackketter Dean 2008-07-22 05:01:48 UTC
On the narrow screen this just doesn't work.

I suggest that we do right/left alignment until there's overlap, then we make it a single scrolling line with a space in the middle.  

As discussed in the call, we should also disable the screensaver while scrolling the first time while interactively browsing.

Signal strength is a very interesting (and special) case, since it needs to dynamically update without scrolling.  The best bet here is to have a short string for the boom display.  How about:  "Wireless signal:"
Comment 7 Adrian Smith 2008-07-25 12:07:06 UTC
Do we want to go back to allowing the font to change for boom?
Comment 8 Adrian Smith 2008-07-25 13:18:19 UTC
see change 22126 - this uses the mixture of right justified and single scrolling line as proposed by Dean.

I've removed the bit which forces the font for Boom as it does not work with measureText at present and decided it looked odd (as per some comments too)
Comment 9 Adrian Smith 2008-08-04 13:27:23 UTC
I've not had any complaints to the my last fix for this so assume everyone is ok with it?  If so will mark as fixed...
Comment 10 Blackketter Dean 2008-08-06 13:20:04 UTC
Looking good.
Comment 11 Spies Steven 2008-08-29 15:26:27 UTC
Verified with SqueezeCenter Version: 7.2 - 22900
Comment 12 James Richardson 2008-12-15 11:58:56 UTC
This bug has been fixed in the latest release of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.