Bug 5052 - Hebrew strings on status screen overlap play icon
: Hebrew strings on status screen overlap play icon
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Localization
: 6.5.2
: PC Windows XP
: P4 minor (vote)
: ---
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-05-14 22:14 UTC by Mickey Gee
Modified: 2008-12-18 11:12 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
Screenshot (3.65 MB, image/bmp)
2007-05-14 22:15 UTC, Mickey Gee
Details
Screenshot of another example (3.65 MB, image/bmp)
2007-05-14 22:18 UTC, Mickey Gee
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mickey Gee 2007-05-14 22:14:12 UTC
Issue found by translators testing Hebrew version of SlimServer. Using either May 12 or May 13 2007 version.

Michael H suggested deferring to post-6.5.2. Reassign to Michael H.

Screenshot appended with issue circled.
Comment 1 Mickey Gee 2007-05-14 22:15:43 UTC
Created attachment 1998 [details]
Screenshot
Comment 2 Mickey Gee 2007-05-14 22:18:02 UTC
Created attachment 1999 [details]
Screenshot of another example
Comment 3 Michael Herger 2007-05-15 01:45:34 UTC
Renaming as the random mix issue is covered by 5044. 
Comment 4 KDF 2007-05-17 00:33:59 UTC
Index: HTML/EN/rtl.css
===================================================================
--- HTML/EN/rtl.css     (revision 12033)
+++ HTML/EN/rtl.css     (working copy)
@@ -43,7 +43,8 @@
 
 div#topGraphicMenu3 #statusSongRemoveButton {
        position: absolute;
-       right: 18px;
+       left: 18px;
+       width: 15px;
 }
 
 div#topGraphicMenu3 #statusMessage {
Comment 5 Michael Herger 2007-05-17 00:42:28 UTC
Kevin, does this work for you in IE?
Comment 6 KDF 2007-05-19 20:43:40 UTC
have to test, but it should work.
Comment 7 KDF 2007-05-20 00:06:01 UTC
better, works on IE and also fixes the location of the player chooser:

Index: HTML/EN/rtl.css
===================================================================
--- HTML/EN/rtl.css     (revision 12055)
+++ HTML/EN/rtl.css     (working copy)
@@ -33,6 +33,10 @@
        left: 20px;
 }
 
+div#topGraphicBox #playerChooser form {
+       float: left;
+}
+
 div#topGraphicMenu1 #powerButtons {
        left: 18px;
 }
@@ -43,9 +47,13 @@
 
 div#topGraphicMenu3 #statusSongRemoveButton {
        position: absolute;
-       right: 18px;
+       left: 18px;
 }
 
+#statusSongRemoveButton img {
+       float: left;
+}
+
 div#topGraphicMenu3 #statusMessage {
        padding-right: 7px;
        float: right;
Comment 8 Michael Herger 2007-05-21 13:42:48 UTC
Change 12066 - thanks Kevin!
Comment 9 Chris Owens 2007-05-29 12:56:00 UTC
Should this be marked as fixed or are we still waiting on some additional change?
Comment 10 KDF 2007-05-29 13:07:32 UTC
should be good.  rtl.css was also merged over to trunk at change 12069, thus includes this fix.