haystack / nb

12 stars 10 forks source link

Allow users to zoomin on PDFs and handle the sidebar properly #159

Open JumanaFM opened 3 years ago

JumanaFM commented 3 years ago

Currently, zooming the pdf also zooms the sidebar which causes the sidebar to go over the text. Need better way to handle this. image

Suggestion: add function to resize the pdf, since using the browser zoom setting will resize the whole page. Another suggestion relates to issue #179 hiding the bar will solve this problem temporarily.

gwtaylor commented 2 years ago

Is this expected behaviour? This is the default view I get on Chrome on MacOS. Standard letter-sized PDF:

Screen Shot 2022-02-24 at 8 57 18 PM

If I use the browser zoom I get fuzzy text. On nb1 the pdf renders and sets default zoom just fine:

Screen Shot 2022-02-24 at 9 00 22 PM

JumanaFM commented 2 years ago

Hello @gwtaylor! Is the browser's zoom set to 100%? If not, try setting the zoom to 100 then refreshing the page, this should give you a better view of the document. Let me know how that goes!

image
gwtaylor commented 2 years ago

Thanks @JumanaFM for the super speedy response. You're the best! This absolutely worked.

I had done a Cmd-0 (reset page zoom) thinking everything was fine. When I explicitly clicked the three dots it reported Zoom 100%, but I was still seeing the page rendered as per my screen shot. The refresh was key. Now I'm seeing expected behaviour. So much better :smile: