Closed Bug 1102063 Opened 6 years ago Closed 6 years ago
"Automatic Zoom" overlaps with arrows in pdf
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.
A work in progress pull request has been submitted at: https://github.com/mozilla/pdf.js/pull/5382.
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.
(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.
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.
We have a contributor working on patches. They should hopefully be up soon.
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
Assignee: nobody → jonas.jenwald
Status: NEW → ASSIGNED
Depends on: 1117757
Comment on attachment 8544152 [details] [diff] [review] bug1102063-zoom-arrows.beta.diff Let's get this up for our final builds.
Whiteboard: [pdfjs-c-ux] → [pdfjs-c-ux] [checkin-needed-aurora] [checkin-needed-beta]
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).
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [pdfjs-c-ux] [checkin-needed-aurora] [checkin-needed-beta] → [pdfjs-c-ux]
Target Milestone: --- → Firefox 37
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.
You need to log in before you can comment on or make changes to this bug.