1102063 - "Automatic Zoom" overlaps with arrows in pdf.js.

Status: VERIFIED FIXED

(Firefox :: PDF Viewer, defect, P2)

Description

[jackwilson128]

Attachment: Screenshot from 2014-11-19 20:09:52.png

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:36.0) Gecko/20100101 Firefox/36.0
Build ID: 20141119184317

Steps to reproduce:

In Firefox nightly 36.0a1 (2014-11-19) I opened a PDF.


Actual results:

At the top of the PDF viewer, the "Automatic Zoom" text overlapped the two arrows.


Expected results:

The text should not overlap the arrows. Perhaps the text should be made narrower or the button made wider.

Comment 1

[Jonas Jenwald [:Snuffleupagus]]

A work in progress pull request has been submitted at: https://github.com/mozilla/pdf.js/pull/5382.

Comment 2

[Loic]

good=2014-10-02
bad=2014-10-03
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=2399d1ae89e9&tochange=b85c260821ab

I'm not sure the bug is in pdf.js, maybe more a layout issue.

Comment 3

[:Gijs (he/him)]

(In reply to Loic from comment #2)
> good=2014-10-02
> bad=2014-10-03
> http://hg.mozilla.org/mozilla-central/
> pushloghtml?fromchange=2399d1ae89e9&tochange=b85c260821ab
> 
> I'm not sure the bug is in pdf.js, maybe more a layout issue.

You're right, this was caused by a change in layout. However, that change was intentional, see comment #1 and its link.

Comment 5

[Lukas Blakk [:lsblakk] use ?needinfo]

Brendan - any chance there's a low risk uplift coming that can prevent us shipping this regression?  We're going to build our final desktop beta this week but if there was something incredibly low risk we could consider it for RC since this is a user-facing and very visible regression.

Comment 6

[Brendan Dahl [:bdahl]]

We have a contributor working on patches. They should hopefully be up soon.

Comment 7

[Jonas Jenwald [:Snuffleupagus]]

Attachment: bug1102063-zoom-arrows.beta.diff

Comment 8

[Yury Delendik (:yury)]

Comment on attachment 8544152 [details] [diff] [review]
bug1102063-zoom-arrows.beta.diff

Approval Request Comment
[Feature/regressing bug #]: Fixes overlap in UI element
[User impact if declined]: UI control/combobox will not look right
[Describe test coverage new/current, TBPL]: manual testing for FF35-37, landed in m-c
[Risks and why]: UI control/combobox will not look right
[String/UUID change made/needed]: n/a

Comment 9

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 8544152 [details] [diff] [review]
bug1102063-zoom-arrows.beta.diff

Let's get this up for our final builds.

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/3e875e5a648d
https://hg.mozilla.org/releases/mozilla-beta/rev/fb06fa0600ab

Fixed on trunk by bug 1117757 (which will be merging to m-c within the next few hours).

Comment 11

[Ada [:adalucinet]]

Reproduced with Nightly 2014-11-16 on Windows 7 64-bit.
Verified as fixed with 35.0 RC (Build ID: 20150105205925), Aurora 36.0a2 (Build ID: 20150106004007) and Nightly 37.0a1 (Build ID: 20150106030201) with various PDFs (embedded, with annotations and outline view) on Ubuntu 12.03 32-bit, Windows 7 64-bit and Mac OS X 10.9.5.