|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
208.34 KB, image/png
59 bytes, text/x-review-board-request
|Details | Review|
295.65 KB, image/png
Created attachment 8835243 [details] linear tabs layout padding errors Some of the paddings in the linear tabs layout aren't set for RTL: there's an outer padding on the thumbnail side that doesn't get reversed in RTL (shows up as too much padding on the left and too little padding on the right in the screenshot), and there are different paddings on the two sides of the title that don't get reversed in RTL (though it's hard to tell visually).
Created attachment 8835261 [details] left side: LTR; right side: RTL, both with the patch Screenshot of LTR and RTL with the patch.
Comment on attachment 8835260 [details] Bug 1338027 - Fix RTL padding in the linear tabs layout. https://reviewboard.mozilla.org/r/110968/#review112314 LGTM! Added Max, who's been working on making Fennec support RTL, as second reviewer.
Comment on attachment 8835260 [details] Bug 1338027 - Fix RTL padding in the linear tabs layout. https://reviewboard.mozilla.org/r/110968/#review112676 LGTM! Thanks, Tom. :)
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/101c4902811d Fix RTL padding in the linear tabs layout. r=maliu,sebastian
Seems to be fixed on latest Nightly, according to Tom's last screenshot.
Max, RTL is landing in 53, right? Should I request uplift to aurora for this and bug 1337897?
Tom, sure do. Both patch have no risk on uplift, please request uplift to aurora. Thank you.
Comment on attachment 8835260 [details] Bug 1338027 - Fix RTL padding in the linear tabs layout. Approval Request Comment [Feature/Bug causing the regression]: RTL for fennec [User impact if declined]: in RTL mode some of the padding on the tabs tray in portrait mode will be wrong (see the first screenshot). [Is this code covered by automated tests?]: No. [Has the fix been verified in Nightly?]: Yes. [Needs manual test from QE? If yes, steps to reproduce]: No I guess. I'm never sure under what conditions I'm supposed to say yes to this :P [List of other uplifts needed for the feature/fix]: None [Is the change risky?]: Low risk [Why is the change risky/not risky?]: The old padding values still exist and haven't changed, we're simply adding extra padding information in the way required by android for RTL. [String changes made/needed]: None
Comment on attachment 8835260 [details] Bug 1338027 - Fix RTL padding in the linear tabs layout. Fix an RTL padding issue in the linear tabs. This polish the UI. Aurora53+.