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

adding GUI similar to pdf.js #505

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

intellisense
Copy link
Contributor

With this pull request the user will have a topbar navigation to perform following actions:

  • Toggle sidebar (toggle side bar button only shown if there is outline)
  • Next/Previous button for navigation
  • Jump to page
  • zoom in and zoom out buttons
  • full screen (presenation mode option)

If user is going to scroll then the page number and outline will be updated/highlighted automatically.
The navbar.js must be included after pdf2htmlEX.js has been included and initialized hence the use of variable pdf2htmlEX inside navbar.js is implicit and clousre compiler raised warning (which we can ignore I guess?)

Final result look like this:

screenshot from 2015-04-11 04 13 13

@@ -7,17 +7,18 @@
/* Part 1: Web Page Layout: Free to modify, except for a few of them which are required by pdf2htmlEX.js, see the comments */
#sidebar { /* Sidebar */
position:absolute;
top:0;
top:32px;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To adjust the fixed top bar nav height.

@coolwanglu
Copy link
Owner

Thanks for the PR, I knew that some (individual) users would like a default viewer, but seems that most users are embedding the output of pdf2htmlEX into their own pages.

The changes in this PR seem intrusive, the output can no longer be embedded this way, and there are likely to be conflicts in CSS rules as common names are used.

It would be best to make an options for this.

@muthugp
Copy link

muthugp commented May 5, 2015

Hi Aamir,
I would like to try your GUI with PDF2HTMLEx output.
I am using Windows build downloaded from http://soft.rubypdf.com/software/pdf2htmlex-windows-version

What should i do to see your GUI and access to navbar.js?

Thanks.

@intellisense
Copy link
Contributor Author

Hi @muthugp I am not sure what has to be done about windows build, sorry.

jwuttke added a commit to jwuttke/pdf2htmlEX that referenced this pull request Sep 29, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants