Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zoom should be centered #7512

Closed
MickL opened this issue Jul 28, 2016 · 4 comments
Closed

Zoom should be centered #7512

MickL opened this issue Jul 28, 2016 · 4 comments

Comments

@MickL
Copy link

MickL commented Jul 28, 2016

Currently when zooming the page-scroll stays the same.
The expected behavior should be a centered zoom to the middle of the viewport. (Example: Google Maps)

A scrolling, according to viewport, needs to be added.

Initial situation:
bildschirmfoto 2016-07-28 um 16 59 48

Current behavior:
bildschirmfoto 2016-07-28 um 17 01 03

Expected behavior:
bildschirmfoto 2016-07-28 um 17 03 59

@Snuffleupagus
Copy link
Collaborator

Snuffleupagus commented Jul 28, 2016

The behaviour you describe is used when zooming is done with a mouse or trackpad (using
Ctrl+scroll wheel/trackpad).

A scrolling, according to viewport, needs to be added.

However, I do not agree that all zooming (e.g. with the toolbar buttons) should behave that way.
Compare with desktop PDF viewers such as e.g. Adobe Reader, which handles zooming similar to PDF.js.
In my opinion, this issue is won't fix.

@MickL
Copy link
Author

MickL commented Jul 28, 2016

To zoom to something specific the user has to do: zoom, scroll, zoom, scroll, ... which is very unhandy.

Similar "big" software that zooms into center on zoom-in-button click:

  • Google Maps
  • Apple Preview
  • Microsoft Image Preview

All of them are much more frequently used than Adobe Reader which lead into the described expected behavior. Im pretty sure additional user-tests would confirm this.

@dasilvacontin
Copy link

I'm having this problem as well. Also, looking forward to adding mobile-zoom via pinch, it's necessary that pdf.js can zoom using any point as the zoom's center.

@Snuffleupagus
Copy link
Collaborator

Also, looking forward to adding mobile-zoom via pinch,

That's already tracked in issue #2582.


Closing as WONTFIX, given the points mentioned in #7512 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants