Horizontal position should also be restored

NEW
Unassigned

Status

()

Firefox
PDF Viewer
P3
normal
5 years ago
5 years ago

People

(Reporter: Dan Jacobson, Unassigned)

Tracking

28 Branch
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [pdfjs-c-ux])

(Reporter)

Description

5 years ago
User Agent: Mozilla/5.0 (X11; Linux i686; rv:28.0) Gecko/20100101 Firefox/28.0 Iceweasel/28.0a2 (Beta/Release)
Build ID: 20131215004001

Steps to reproduce:

With user_pref("browser.startup.page", 3);
browse a local PDF and hit CTRL++ to zoom text enough until horizontal scrolling is caused,
and then scroll horizontally a bit. Now hit CTRL+Q and then start firefox again. 


Actual results:

Vertical position is restored, but not horizontal position.



Expected results:

Horizontal position should also be restored.
I only checked with a local PDF. Maybe affects more than just PDF, and more than just local files.
(Reporter)

Updated

5 years ago
Component: Untriaged → PDF Viewer
This should already be fixed upstream: https://github.com/mozilla/pdf.js/pull/3967.
Mozilla/5.0 (X11; Linux x86_64; rv:29.0) Gecko/20100101 Firefox/29.0

Tested on latest Nightly (20140106030203) - the horizontal position is not remembered in a regular pdf.
Does the fix include also this? Thank you
Flags: needinfo?(jonas.jenwald)
(In reply to petruta.rasa from comment #2)
> Mozilla/5.0 (X11; Linux x86_64; rv:29.0) Gecko/20100101 Firefox/29.0
> 
> Tested on latest Nightly (20140106030203) - the horizontal position is not
> remembered in a regular pdf.

The patch hasn't been uplifted to mozilla-central yet, but I expect it to be included in the next PDF.js update.

> Does the fix include also this? Thank you

Yes. The patch added the necessary code for horizontal scrolling to work in general, so this bug is one issue that was fixed by it.
Flags: needinfo?(jonas.jenwald)
(Reporter)

Updated

5 years ago
Duplicate of this bug: 956078
Priority: -- → P3
Whiteboard: [pdfjs-c-ux]
Marking New as per Comment 3.
Status: UNCONFIRMED → NEW
Ever confirmed: true
You need to log in before you can comment on or make changes to this bug.