Skip to content
This repository has been archived by the owner on Dec 9, 2018. It is now read-only.

Page up/page down do not work reliably #180

Open
dbdr opened this issue Jul 2, 2013 · 4 comments
Open

Page up/page down do not work reliably #180

dbdr opened this issue Jul 2, 2013 · 4 comments

Comments

@dbdr
Copy link
Contributor

dbdr commented Jul 2, 2013

When an html create by pdf2htmlEX is loaded, page up and page down do not work (observed on both FF and Chrome). If I click on text, then it does start to work, but it stops after a few presses, until I click again.

I see this on most/all documents, which are usually big and some are confidential, so I don't attach an example right now, but let me know if you have trouble reproducing.

And cheers for your work, the functionality is amazing!

@coolwanglu
Copy link
Owner

Yes, I'm aware of this. It's mostly because the height of the pdf container is fixed, so technically the browser does not see anything outside the screen.

Usually I don't work on UI stuffs, but I will be glad to merge if you can create a patch.

@dbdr
Copy link
Contributor Author

dbdr commented Jul 3, 2013

I see. I might give it a try. Any idea, what kind of change is needed for this to work?

@coolwanglu
Copy link
Owner

I guess a few lines of JavaScript should work.

@coolwanglu
Copy link
Owner

Although #181 is merged, it is not enabled by default. I think I'll add a new parameter for this.

william-bratches pushed a commit to william-bratches/CourseScraper that referenced this issue May 6, 2014
Add key handler, and implement page up, page down, ctrl-home and ctrl-end.

First commit on github and first javascript coding :)
william-bratches pushed a commit to william-bratches/CourseScraper that referenced this issue May 6, 2014
Better name for the key handler.
william-bratches pushed a commit to william-bratches/CourseScraper that referenced this issue May 6, 2014
Call e.preventDefault() when the key was handled.
william-bratches pushed a commit to william-bratches/CourseScraper that referenced this issue May 6, 2014
Scroll pages with alt-page up/down, not page up/down.

Implement screen scrolling for page up/down.

Fix comment about which key is up (code was right, but comments were inverted :D)
william-bratches pushed a commit to william-bratches/CourseScraper that referenced this issue May 6, 2014
william-bratches pushed a commit to william-bratches/CourseScraper that referenced this issue May 6, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants