Page MenuHomePhabricator

Fit scanned page portion within the hieght of the screen while using PagelistWidget
Closed, ResolvedPublic

Description

While using PagelistWidget, the scanned lower part of the page remains outside of the screen, which leads to wasting of time to drag every page. It would be more helpful if the scanned page remains within the height of the screen.

Screenshot from 2021-08-15 10-45-42.png (902×1 px, 1 MB)

Event Timeline

This should be resolved by just remembering the zoom and pan position (we used to remember the scroll position before). Trying to show the whole page (which is the default for Openseadragon) is going to make the text too small to read on smaller screens.

Agreed that it would be smaller than now, but that does not prevent indexing for majority of the pages. We just need to get a glimpse of what the page is about, whether it's a numbered page, a cover page, a blank page etc. We don't have to read the whole page to understand it. Fitting into screen will give us the whole idea about the page without scrolling down everytime. Also, it would take much lesser time to load than what is happening now. We don't have to zoom and pan all pages, but If we need to do that for some of the pages, you are going to install that feature anyway. So there is no conflict in this request with that.

Change 714193 had a related patch set uploaded (by Yash9265; author: Yash9265):

[mediawiki/extensions/ProofreadPage@master] Allowing to zooming out the image Adding the min zoom level and max zoom level which will help to zoom out and zoom in the image and also preserving the viewport for remembering the zoom and pan.

https://gerrit.wikimedia.org/r/714193

Change 714193 merged by jenkins-bot:

[mediawiki/extensions/ProofreadPage@master] Allowing to zooming out the image

https://gerrit.wikimedia.org/r/714193

@Bodhisattwa, Can you take a look and see if the current behaviour suits the use-case you were talking about (demo on testwikisource). We have essentially removed a parameter that set the minimum zoom level to the default zoom level and enabled remembering of zoom and pan position between pages.

@Bodhisattwa, Can you take a look and see if the current behaviour suits the use-case you were talking about (demo on testwikisource). We have essentially removed a parameter that set the minimum zoom level to the default zoom level and enabled remembering of zoom and pan position between pages.

It stll doesn't fit within the height of the screen.

Screenshot from 2021-08-23 10-24-00.png (905×1 px, 389 KB)

@Bodhisattwa, Can you take a look and see if the current behaviour suits the use-case you were talking about (demo on testwikisource). We have essentially removed a parameter that set the minimum zoom level to the default zoom level and enabled remembering of zoom and pan position between pages.

It stll doesn't fit within the height of the screen.

Screenshot from 2021-08-23 10-24-00.png (905×1 px, 389 KB)

@Bodhisattwa If you pan up a bit, I think you should be able to see the whole page (the controls are zoom in zoom out using the scroll wheel/arrows on top, drag image with mouse to reposition/pan).

@Bodhisattwa If you pan up a bit, I think you should be able to see the whole page (the controls are zoom in zoom out using the scroll wheel/arrows on top, drag image with mouse to reposition/pan).

Yes, its manageable now.