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

added view button to view pdf&epub in the browser #374

Merged
merged 3 commits into from
Feb 25, 2018

Conversation

rockonedege
Copy link
Contributor

Now that major browsers support viewing pdf(Firefox&Edge) and epub(Edge and Firefox w/ extension) files directly from inside the browsing window, it's my own needs to leverage this functionality for reading ebooks on phones. It might be helpful to others as well.

It's my first time to touch PHP code. Feel free change it.

@seblucas
Copy link
Owner

Hi,

Thanks for this PR, it seems mostly good. I'll check that out locally and get back to you.

@seblucas
Copy link
Owner

As a side note, EPUB support on Edge is very picky (many epub I own despite being perfectly valid are not read by Edge)

@rockonedege
Copy link
Contributor Author

Yes, the EPUB support on Edge is also slow when the file gets large. Hopefully it will get better as it's heavily invested by Microsoft currently.

@seblucas seblucas merged commit 042217e into seblucas:master Feb 25, 2018
@seblucas
Copy link
Owner

Sorry for the delay and thanks again

seblucas added a commit that referenced this pull request Feb 25, 2018
Add icons to make things easier to handle.
Add hints to all button.
Don't limit to PDF / EPUB, if someone has a FB2 / Mobi reader in its browser / PC let it use it
@marioscube
Copy link

Downloading (map icon) does not work on symlinked libraries, only on real paths.

@seblucas
Copy link
Owner

seblucas commented Apr 7, 2018

Hi Mario,

What do you mean ? Is there a difference between the download and the open icons ?

I never used a symlink for my libraries but I see no reason for a discrepancy here.

@marioscube
Copy link

Hi Sebastien,

Normally I see no difference. However on a symlinked library he "epub" icon works nomally, but the "map" icon gives a "file not found".

I use firefox on the desktop. The same error with webkit on iOS.

seblucas added a commit that referenced this pull request Oct 9, 2018
The PR #374 was badly tested by me and did not include anything for URL Rewriting :(.
dunxd pushed a commit to dunxd/mikespub-cops that referenced this pull request Sep 10, 2023
Add icons to make things easier to handle.
Add hints to all button.
Don't limit to PDF / EPUB, if someone has a FB2 / Mobi reader in its browser / PC let it use it
dunxd pushed a commit to dunxd/mikespub-cops that referenced this pull request Sep 10, 2023
…eblucas#374

The PR seblucas#374 was badly tested by me and did not include anything for URL Rewriting :(.
dunxd pushed a commit to dunxd/mikespub-cops that referenced this pull request Sep 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants