Bug 8622 - UI Bug Box for player selection not displayed correctly
: UI Bug Box for player selection not displayed correctly
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.1
: PC Windows Vista
: P2 normal (vote)
: 7.x
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-07-01 13:09 UTC by Johan Sombekke
Modified: 2009-07-31 10:23 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
Combobox not correctly displayed (4.44 KB, image/jpeg)
2008-07-01 13:09 UTC, Johan Sombekke
Details
possible fix? (48.18 KB, text/plain)
2008-07-02 00:44 UTC, Michael Herger
Details
improved patch (48.17 KB, text/plain)
2008-07-02 00:47 UTC, Michael Herger
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Johan Sombekke 2008-07-01 13:09:24 UTC
Created attachment 3520 [details]
Combobox not correctly displayed

Minor UI bug. Whenever there's more than one player the dropdown box to choose a player is displayed 'wrapped up'. Please see attachment. I'm using IE 7 on Windows Vista.

PS This problem has occured on all builds so far, so is not 'build' related.
Comment 1 Michael Herger 2008-07-01 23:48:26 UTC
That's interesting (and annoying, too...) to know, as I've rewritten almost all JS for 7.1 :-/.

Do you see the size change at some moment or is it that small from the beginning? Would a reload work?
Comment 2 Michael Herger 2008-07-02 00:37:36 UTC
One small test: what does the following return if you enter it in your IE7's address field?

javascript:alert(document.compatMode)
Comment 3 Michael Herger 2008-07-02 00:40:06 UTC
Wasn't exact enough: run that command from the address field once you've loaded SC in that browser window (eg. replace "http://localhost:9000" with the above)
Comment 4 Michael Herger 2008-07-02 00:44:40 UTC
Created attachment 3524 [details]
possible fix?

And then... could you please replace HTML/EN/html/SqueezeJS/UI.js with this attached version? It implements a workaround I've found in the ExtJS forums for a similar issue http://extjs.com/forum/showthread.php?t=23969
Comment 5 Michael Herger 2008-07-02 00:47:04 UTC
Created attachment 3525 [details]
improved patch
Comment 6 Michael Herger 2008-07-02 00:58:37 UTC
*** Bug 7457 has been marked as a duplicate of this bug. ***
Comment 7 Johan Sombekke 2008-07-02 04:50:48 UTC
Hi Michael,

To answer your questions:
- The size changes only when there's more than one SB and if that's the case only when a different SB is choosen (so, when you don't touch the button, nothing will happen after adding another SB)
- on entering: javascript:alert(document.compatMode) it says: CSS1 Compat
- unfortunately your patch doesn't solve the problem. So: the box displays properly on initial load or reload of squeezecenter, but as soon as I choose another player the box' size changes.
Comment 8 Michael Herger 2008-07-02 05:14:56 UTC
Thanks a lot! Your hint about choosing another player at last allowed me to reproduce this issue. Obviously I never used IE7 to choose a player... 

Should be fixed with change 21425 - don't clear the player name before setting the new one, as IE7 would miscalculate the button's width.
Comment 9 Chris Owens 2008-07-30 15:31:27 UTC
This bug has now been fixed in the 7.1 release version of SqueezeCenter!  Please download the new version from http://www.slimdevices.com 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.
Comment 10 Chris Owens 2009-07-31 10:23:33 UTC
Reduce number of active targets for SC