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

Document zoom level often too low; document would still fit with one or two notches higher zoom #23

Open
jstray opened this issue Jan 11, 2013 · 2 comments

Comments

@jstray
Copy link

jstray commented Jan 11, 2013

This happens with many documents and browser widths, but for a concrete example: set your browser window width to 1110 (or thereabouts, check in console with window.innerWidth), then go to https://www.documentcloud.org/documents/527670-oregon-zoo-elephant-contract.html

(Also make sure your browser is at default zoom and that the sidebar appears.)

The zoom level that the document loads at is lower than it could be. On Chrome/Mac, you could up the zoom by two notches and still contain the entire document image plus the "p. 1" text. On Safari/Mac you could increase by one notch.

Getting the image as large as possible is important for Overview, because the user relies in part on very rapid user scanning of many documents, and we don't have any screen real estate to waste on needless padding.

@jashkenas
Copy link
Member

The maximum zoom level by default (unless you override it) is the 700px-wide version of the image. Anything larger than that needs to load the 1000px-wide versions of the image. Part of our rationale was not making documents unintentionally comically large. Some folks have big monitors, and the full 1000px zoom can look like old-folks font.

That said, I think you have a good point for many documents with fine print. Let's change the logic to allow the "autozoom" feature even at sizes larger than 700px.

@jstray
Copy link
Author

jstray commented Jan 16, 2013

Sweet! Thanks so much, this will make a noticeable difference in workflow. Right now it's a drag to drag the zoom on every doc.

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

No branches or pull requests

2 participants