1059845 - float: right; does not position buttons right in the pdf viewer toolbar

Status: RESOLVED DUPLICATE of bug 1059167

(Core :: Layout, defect)

Description

[Yury Delendik (:yury)]

Attachment: Screen Shot 2014-08-28 at 9.46.22 AM.png

Breaks toolbar buttons layout in the pdf viewer when RTL languages are used. See the attachment. For testing http://mozilla.github.io/pdf.js/web/viewer.html#locale=ar can be used.


Bisecting nightlies:

Last good revision: daa84204a11a (2014-08-25)
First bad revision: dc352a7bf234 (2014-08-26)
Pushlog:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=daa84204a11a&tochange=dc352a7bf234


Bisecting inbounds:

Last good revision: daa84204a11a
First bad revision: 156f1bee4139
Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=daa84204a11a&tochange=156f1bee4139


Bisecting via build:

The first bad revision is:
changeset:   201330:54ada5ad66bb
user:        Jonathan Kew <[e-mail]>
date:        Sun Aug 24 15:34:44 2014 +0100
summary:     bug 1046950 pt 2 - convert ComputeSize to use logical-coordinate parameters. r=smontagu

Comment 1

[Yury Delendik (:yury)]

Attachment: Minimal test case to reproduce the issue

Comment 2

[Yury Delendik (:yury)]

[Tracking Requested - why for this release]:
PDF Viewer UI regression for some cultures

Comment 3

[Boris Zbarsky [:bzbarsky]]

Duplicate of bug 1059167, no?

Comment 4

[Yury Delendik (:yury)]

(In reply to Boris Zbarsky [:bz] from comment #3)
> Duplicate of bug 1059167, no?

yes, patch from bug 1059167 fixes the issue