Closed Bug 1164419 Opened 5 years ago Closed 5 years ago
[OSX] Pocket panel for ru locale build has misaligned elements
Reproducible with Firefox 38.0.5 beta 1 build 2, ru locale. Str: 1. Login to Pocket and load a page in a new tab 2. Select the Pocket toolbar icon to add the page to Pocket list ER: A message is displayed to inform the user that the page was added. Tags can be added to that page. Everything looks correctly, no scrollbar AR: The elements are not properly aligned. Scrollbar is visible. Please see the attachment.
[Tracking Requested - why for this release]: This looks bad, we should fix this or disable Russian.
Hmm, the May 9th code drop () improved localized layout, but it wasn't this broken before or after in screenshots of Pocket's testing of that drop. https://www.dropbox.com/s/tw9ju4x1cxg0yxm/Screenshot%202015-05-13%2011.43.07.png?dl=0
I can reproduce with a build from https://ftp.mozilla.org/pub/mozilla.org/firefox/candidates/38.0.5b1-candidates/build2/mac/ru/
The issue was an unexpected css selector on the panel setting the max-width of the panel to 30em. This made the saved panel 8px smaller than it should have been which broke the layout for the much longer ru strings. The offending selector was: http://mxr.mozilla.org/mozilla-central/source/browser/themes/shared/customizableui/panelUIOverlay.inc.css#244 Dolske suggested the new selector that is in this patch (we worked through it live on IRC together, I'm just submitting the patch file).
I made the last part of the selector a little stricter (matching the original one above it), just to be extra safe. (I also verified that it's still wider and makes the RU panel contents fit correctly.)
Comment on attachment 8605595 [details] [diff] [review] Patch v.2 [Triage Comment] a+ for aurora/beta/release: required for Pocket launch in 38.0.5.
Verified as fixed using Firefox 38.0.5 beta 3 under Mac OS X 10.9.5.
You need to log in before you can comment on or make changes to this bug.