Bug 11606 - Error strings on System Diagnostic screen too long
: Error strings on System Diagnostic screen too long
Status: CLOSED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: Diagnostics
: unspecified
: PC Windows XP
: P1 normal (vote)
: 7.4.0
Assigned To: Ben Klaas
: SLT
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-04-01 15:38 UTC by Mickey Gee
Modified: 2009-10-05 14:30 UTC (History)
5 users (show)

See Also:
Category: ---


Attachments
Ref artwork - stacked compare (86.14 KB, image/png)
2009-08-19 10:25 UTC, ndijulio
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mickey Gee 2009-04-01 15:38:41 UTC
In reviewing the System Diagnostic screen viewable during the Setup process, most strings are either left or right justified on the screen. As a result, if a string is too long then it will overlap a string justified on the other edge of the screen which is on the same line.

In this case, the strings that are too long do not need rewrite but may simply require use of another string that's previously translated.

The too long strings are usually for the following:

mysqueezebox.com:  DNS failed
  Ping:            Can't ping         (#1)
  TCP port 3483:   Can't connect      (#2)
  TCP port 9000:   Can't connect      (#3)

The long strings appear to overly verbose in describing the above status. One solution would be to shorten these strings. Another solution would be say the following:

  Ping:            Not connected
  TCP port 3483:   Not connected
  TCP port 9000:   Not connected

"Not connected" is the phrase used for describing SqueezeCenter connection errors:

SqueezeCenter:     Not connected
  Ping:            Not connected
  TCP port 3483:   Not connected
  TCP port 9000:   Not connected

For some reason, "Not connected" in other languages fits pretty well.

Here are some of the strings I found that could use shortening by replacing with the "Not connected" string:

Danish

#2, #3: Der kan ikki etableres forbindelse
Replace with Ikke tilsluttet

German

#2, #3: Berbindung kann nicht hergestellt werden"
Replace with Nicht verbunden

French

#1: "Impossible d'executer la commande PING"
Replace with Non connecte
#2,#3: Connextion impossible
This one fits, but I don't understand why you'd want to say "Connection not possible" instead of "Not Connected".

Dutch

#2, #3: Kan geen verbinding maken
Replace with "Niet verbonden"

Polish

#2,#3: Nie mozna nawiazac polaczenia
Replace with Niepolaczony

Russian:

#2,#3 have issue (don't have Cyrillic keys on my keyboard!)
Looks like replacing with Russian for "Not connected" would work here too.

English, Finnish, Czech, Spanish, Norwegian, Italian are OK

System Diagnostics screen can be reached by pressing the Help button the Choose Wireless Network screen, then the More button on the upper right of the next screen, and then System Diagnostic at the bottom of the "More Help" screen.
Comment 1 Mickey Gee 2009-04-01 15:41:18 UTC
Seen in firmware r5072.
Comment 2 Blackketter Dean 2009-04-02 16:00:00 UTC
How about just omitting lines for tests that can't be run (i.e. don't show the IP address line when there is no network connection, don't show the Ping line for SN when DNS fails.
Comment 3 Blackketter Dean 2009-04-02 16:13:23 UTC
Ah, this is just a localization issue.  My comment would be a separate bug.
Comment 4 Michael Herger 2009-04-08 09:59:19 UTC
Mickey/SLT - I've applied Mickey's proposals on our side. Please update in your translation memories.

For the next pass we should try to get translations which aren't longer then the English.
Comment 5 Ben Klaas 2009-04-13 10:45:25 UTC
SLT Team- there are currently four bugs assigned to you that have a target of "MP". The deadline for that milestone is this Wednesday April 15th. Could you update us on whether we can expect updated translations on these bugs by that date?
Comment 6 Michael Herger 2009-04-14 02:22:46 UTC
Ben - please see my previous comment. I've already applied the changes in SVN. SLT only needs to update their translation memories on their side.
Comment 7 Ben Klaas 2009-04-15 07:45:55 UTC
given Michael's last comment, this is no longer an MP target bug
Comment 8 Michael Herger 2009-08-11 06:02:44 UTC
Oh well, they're now longer than ever. We'll never manage such issues before we have a means to give SLT/translators some information about our strings. I'll work something out post 7.4
Comment 9 SVN Bot 2009-08-11 08:05:02 UTC
 == Auto-comment from SVN commit #7006 to the jive repo by michael ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7006 ==

Bug: 11606
Description: add hint for translators
Comment 10 Weldon Matt 2009-08-18 17:04:55 UTC
Arrggh.

Nver saw this bug until now.

Why can't these just scroll just like the behavior on other menus?
Comment 11 Weldon Matt 2009-08-18 17:08:19 UTC
To clarify:

The text items should scroll as a single block of text (when you select one of them).  On fab4 this would require a press, on other UIs it would not.
Comment 12 Michael Herger 2009-08-18 23:41:20 UTC
Matt - this indeed isn't a SLT bug any more. With the larger font that screen is almost unusable on the Controller. The MAC address is overlapping the title.

Re. Scrolling: right now the left part (title) is scrolling, while the value is always visible. This imho makes sense, as the right part ist the information we want to see. Having to wait until the full line has scrolled, then scrolling eg. the IP address off the screen while you're writing it down would be very frustrating.

This goes along the line of my complaint about the main menu items being cut off. I'm all for consistency, and I agree that Baby might need larger fonts. But on the Controller's portrait mode screen the larger fonts renders some of the screens very, very hard to read. 

Let's make it consistently not suck instead of using a consistent font size across all platforms ;-).

Trying to get a screenshot.
Comment 13 ndijulio 2009-08-19 09:46:24 UTC
I agree with Micheal here in regards to scrolling items.  A simple enhancement would be to increase the padding when the left text scrolls away from the fixed right text.  Right now the two almost overlap = not very legible.  This is another case where mixed font styles or color would help legibility.

Another way to tackle this would be to look into a stacked text version (similar to albums with no icon).  Will attach screen comps in a few...
Comment 14 ndijulio 2009-08-19 10:25:34 UTC
Created attachment 5630 [details]
Ref artwork - stacked compare
Comment 15 Weldon Matt 2009-08-19 11:03:09 UTC
(In reply to comment #14)
> Created an attachment (id=5630) [details]
> Ref artwork - stacked compare

I think this is great.. any objections?
Comment 16 Mickey Gee 2009-08-19 11:20:47 UTC
Looks great. I like the one with the values in larger bold font. We should make it easier for the customer to read his diagnostic values to Support.
Comment 17 Michael Herger 2009-08-19 22:09:15 UTC
> Looks great. I like the one with the values in larger bold font.

Seconded! Great improvement!
Comment 18 Richard Titmuss 2009-08-20 14:23:34 UTC
Agreed, one on the right is good. Assigning to Ben (or Tom) to implement.
Comment 19 SVN Bot 2009-08-24 14:22:29 UTC
 == Auto-comment from SVN commit #7244 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7244 ==

Fixed Bug: 13472
Fixed Bug: 11606
Description: add item_info style to QVGA skins (still needs adding to WQVGA, bug 13460)
add setText() method to SimpleMenu
modify strings to accomodate two line label for diag items
tweak top/bottom padding of icon_list items
move away from using Choice widgets in diagnostics to using two lines of text in a label
change setValue() method to deal with change in widgets
Comment 20 Ben Klaas 2009-08-24 14:34:04 UTC
Changing hours worked to reflect time spent on this bug, which was much more involved than originally thought.
Comment 21 ndijulio 2009-09-14 10:58:12 UTC
Note: need to update fab4 UI as well.  Current build r7547 has both lines bold and same size font.  

See albums template for correct size and fonts.  Please note that this is a modified version as the text is flipped:

1. Title: FS Med. (top line)
2. Subject: FS Bold (bottom line)
Comment 22 James Richardson 2009-10-05 14:30:45 UTC
This bug has been marked as fixed in the 7.4.0 release version of SqueezeBox Server!
    * SqueezeCenter: 28672
    * Squeezebox 2 and 3: 130
    * Transporter: 80
    * Receiver: 65
    * Boom: 50
    * Controller: 7790
    * Radio: 7790  

Please see the Release Notes for all the details: http://wiki.slimdevices.com/index.php/Release_Notes

If you haven't already, please download and install the new version from http://www.logitechsqueezebox.com/support/download-squeezebox-server.html

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