search

izuzak / atom-pdf-view

Support for viewing PDF files in Atom.
https://atom.io/packages/pdf-view
MIT License
106 stars 30 forks source link

Set initial zoom level to fit width of current buffer #73

Closed afischer closed 8 years ago

afischer commented 9 years ago

It would be fantastic if pdf-view could set the initial zoom so that the PDF's width fit the width of the pane it is being opened in, preventing sideway scrolling.

izuzak commented 9 years ago

That's a really neat idea, @afischer -- I'd definitely like to see that ability in this package (but probably behind a config setting so that it doesn't disrupt someone's workflow).

I don't think I'll be able to dedicate time to this in the near future, but happy to review+merge a pull request if someone would like to tackle this.

Thanks for suggesting this :star2:

michal-szostak commented 8 years ago

I was also going to suggest this feature, but I see that someone thought about it earlier. I would actually recommend to give an option (maybe global one?) to enable / disable this kind of behaviour (for small screens it's very welcomed (as original size of the document usually doesn't fit) but for bigger ones it may not be so.

YarnSphere commented 8 years ago

:+1:

izuzak commented 8 years ago

pdf-view 0.41.0 now has a config option for this, thanks to @nsaje :zap:

Thanks again for suggesting this