Skip to content
This repository has been archived by the owner on Jun 27, 2020. It is now read-only.

[Error] PDF cover cannot be shown #35

Open
seuplus opened this issue Jan 6, 2018 · 11 comments
Open

[Error] PDF cover cannot be shown #35

seuplus opened this issue Jan 6, 2018 · 11 comments

Comments

@seuplus
Copy link

seuplus commented Jan 6, 2018

Platform: Debian GNU/Linux 8.10 (jessie)

1、[error] PDF cover cannot be shown correctly. See the Picture1 below.
2、[enhancement] Can you add a go-back-to-cover button on epub browser. See the Picture2 below.

Thank you!

Picture1
pdf-cover-error

Picture2
screen shot 2018-01-06 at 19 22 40 1 1

@seuplus seuplus changed the title PDF cover cannot be shown [Error] PDF cover cannot be shown Jan 6, 2018
@pgaskin
Copy link
Owner

pgaskin commented Jan 6, 2018

For the pdf cover, currently there is no way to extract it (there are no libraries written in go) short of writing a whole pdf rendering library (which would take a lot of work) or calling an external command (which will greatly decrease the indexing speed, and will not work on all systems). If I find a PDF library written in go, I will add covers.

For your second issue, it actually belongs in http://github.com/geek1011/ePubViewer/issues . You can view the cover by pressing the info button in the top left, or you can go to it in the table of contents. As for clicking the book title, may I ask why this would be useful?

@seuplus
Copy link
Author

seuplus commented Jan 7, 2018

Thank you!

@seuplus seuplus closed this as completed Jan 7, 2018
@LiMinggang
Copy link

Hi, @geek1011, is it possible to use the basic pdf reader implementation like https://github.com/rsc/pdf which lacks of rendering part, or https://github.com/yob/pdfreader which can convert pdf to svg.
Or, is it possible to use front-end pdf reader to do this?
It's very helpful for the pdf books.

@pgaskin
Copy link
Owner

pgaskin commented Dec 27, 2018

Maybe. I need to look into this more.

@LiMinggang
Copy link

LiMinggang commented Dec 28, 2018

Please make high priority to issue #47 which is a big performance improvement for huge books like mine. I don't think the projects I listed are mature enough for use.
And I think I will try a workaround for the thumbnail issue of PDF, ie, use ghost script to generate thumbnails for BookBrowser. Anyway, the books should not update too often.

@pgaskin
Copy link
Owner

pgaskin commented Dec 29, 2018

Yep. I was thinking of something like that. I'll probably make it call imagemagick if it is present.

As for #47, I'm working on it when I have time. I've figured out how to do an efficient checksum of the epub without reading the whole thing, and now, I just need to figure out how I'm going to store metadata in a forwards compatible way.

@LiMinggang
Copy link

Another candidate in go
https://github.com/hhrutter/pdfcpu

@pgaskin
Copy link
Owner

pgaskin commented Jan 3, 2019

@LiMinggang That looks almost perfect! I'll see how it works later this week.

@LiMinggang
Copy link

There is another JS version which supports PDF thumbnail. No idea if it's OK for us.
https://github.com/oguzhaninan/Buka

@pgaskin
Copy link
Owner

pgaskin commented Jan 4, 2019

No, that won't work. It uses PDF.js to render it. Last time I tried, I couldn't get it running in otto (a js runtime for go).

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

3 participants