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

Generate page previews even more lazily #815

Closed
jdm opened this issue Nov 17, 2011 · 6 comments
Closed

Generate page previews even more lazily #815

jdm opened this issue Nov 17, 2011 · 6 comments

Comments

@jdm
Copy link
Contributor

jdm commented Nov 17, 2011

I have a 74 page paper from the ACM open in the FF addon (FF nightly), and if I happen to mouse over the sidebar such that it opens, generating the previews for all 74 pages slows FF to a standstill for at least a minute. It's hard to tell who is at fault here, but I think that the best solution is to generate previews as needed - if I don't scroll the list (and in fact close the sidebar), there shouldn't be any need to generate any previews that aren't in view.

@joneschrisg
Copy link
Contributor

By "slows to a standstill", do you literally mean unresponsive, or very slow?

@jdm
Copy link
Contributor Author

jdm commented Nov 17, 2011

It beachballs for seconds at a time, and the UI is only active in short bursts in between.

@joneschrisg
Copy link
Contributor

Sounds like we're not interrupting main-thread canvas drawing frequently enough. @jviereck @brendandahl

@jviereck
Copy link
Contributor

I've opened #817 to tackle the "don't freeze the UI during canvas rendering" thing. The "render preview not from 1 to X" is not part of that issue.

@brendandahl
Copy link
Contributor

I think there's two problems:

  1. The sidebar currently isn't very "smart" and it starts from the first page and starts rendering every single preview.
  2. We need to fiddle with the drawing interrupt timing and probably look into doing a waited counter like poppler does.

There's more on 1 in issue #780

@brendandahl
Copy link
Contributor

I believe this has been resolved with #854. If I missed something feel free to leave a comment and I'll re-open the issue.

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

No branches or pull requests

4 participants