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

[Bug]: The rendering indicator is not always visible #19130

Open
nicolo-ribaudo opened this issue Nov 29, 2024 · 2 comments
Open

[Bug]: The rendering indicator is not always visible #19130

nicolo-ribaudo opened this issue Nov 29, 2024 · 2 comments
Labels

Comments

@nicolo-ribaudo
Copy link
Contributor

Attach (recommended) or Link to PDF file

This is not specific to any PDF file.

Web browser and its version

Firefox 133

Operating system and its version

Not relevant

PDF.js version

master (308ca2a)

Is the bug present in the latest PDF.js version?

Yes

Is a browser extension

No

Steps to reproduce the problem

  1. Open https://www.gtt.to.it/cms/risorse/urbana/mappa/mapparete.pdf
  2. Set the zoom to 200%

What is the expected behavior?

While PDF.js is rendering pages, it shows a little loading spinner, that disappears once the page is done rendering.

The steps above should show a loading spinner.

What went wrong?

The loading spinner is added to the document, however it's added to the center of the page. The center of the page is outside of the browser viewport, so the spinner is not visible.

Link to a viewer

No response

Additional context

I noticed this while working on #19128. The spinner should be in the middle of the visibleArea computed in that PR.

@nicolo-ribaudo nicolo-ribaudo changed the title [Bug]: The rendering spinner is now always visible [Bug]: The rendering spinner is not always visible Nov 29, 2024
@nicolo-ribaudo nicolo-ribaudo changed the title [Bug]: The rendering spinner is not always visible [Bug]: The rendering indicator is not always visible Nov 29, 2024
@Snuffleupagus
Copy link
Collaborator

Snuffleupagus commented Nov 29, 2024

The spinner in the pageNumber-input (on the toolbar) is however always visible, which makes this a lot less problematical (and possibly even a non-issue).

@nicolo-ribaudo
Copy link
Contributor Author

nicolo-ribaudo commented Nov 29, 2024

Not on Firefox for Android, since the toolbar is not shown there :)

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

No branches or pull requests

3 participants