Open Bug 1171659 Opened 9 years ago Updated 2 years ago

PDF Viewer: Ctrl + scroll wheel behavior is different to Ctrl + [+/-]

Categories

(Firefox :: PDF Viewer, defect, P3)

defect

Tracking

()

People

(Reporter: arni2033, Unassigned)

References

(Blocks 1 open bug, )

Details

(Whiteboard: [pdfjs-ux])

Attachments

(3 files)

Ctrl+[+/-] changes either pdf scale or page scale by ~10% (see bug 786674), but
Ctrl+Wheel[Up/Down] changes pdf scale to maximum/minimum

See attachment videos and bug 750658 that was somehow marked "WorksForMe"
Reposting video of how Ctrl+[+/-] acts
I was unable to reproduce with my mice (microsoft and some really old one) and thinkpad's trackpads. 

Looks like it depends on some mouse (or firefox) setting. Can you check if you can reproduce the issue with new profile? Also, can you add a description of your mouse and its settings (if you changed any)?
Flags: needinfo?(arni2033)
Of course it's reproducible on clear profile.
Mouse: a4tech, Model: OP-720

I also found out that if I set mouse options to "Scrolling the wheel is to scroll N lines", then Ctrl+ScrollWheelUp/Down in PDF viewer would act like "Ctrl+[+/-]" pressed N times. If N=1 then everything would work as expected.
But I always use mouse option "Scrolling the wheel is to scroll 1 screen". And it works as expected when scaling Up/Down the regular html-pages. But not with PDFs.
Flags: needinfo?(arni2033)
So, I figured out what was causing this issue.
The code in PDFJS is supposed to press Ctrl+[±] N times, where N is -event.detail on 'DOMMouseScroll' event, but according to this link, when you choose "scroll by 1 page" on Windows, then event.detail is set to ±32768:
https://developer.mozilla.org/en-US/docs/Web/Events/DOMMouseScroll#detail

Issue I opened on github: https://github.com/mozilla/pdf.js/issues/6263
Status: UNCONFIRMED → NEW
Ever confirmed: true
Has STR: --- → yes
Priority: -- → P3
Whiteboard: [pdfjs-c-ux]
See Also: → 1251987
See Also: → 1253484
See Also: 1251987
Whiteboard: [pdfjs-c-ux] → [pdfjs-ux]
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: