-
Notifications
You must be signed in to change notification settings - Fork 255
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
Huge file size #37
Comments
Also seeing this. |
Hi both, thanks for reporting. There's an issue open on phantomjs (the module that renders the PDF from HTML) so hopefully it'll be fixed soon: ariya/phantomjs#10373 |
Thanks - do you have any views on when that's likely to happen? That issue's been open for 2 years now, and the latest thought is PhantomJS 2 might fix it, but we don't have a date for that. Have you tested with a PhantomJS 2 build? |
+1 Same here. Enormous file and unselectable text. |
For what it's worth, PhantomJS on Windows produces texty PDFs instead of just images. |
Same problem. 15k markdown results in 4.8 MB PDF. The problem seems to be that the fonts are converted to path. This is a showstopper for me. Any progress on this? |
This is an OS-X-only PhantomJS issue. This PR ariya/phantomjs#13243 looks like a fix. People on OS X, could you test and recommend upstream to merge it? |
I'm not optimistic about ariya/phantomjs#13243. I ran some tests with https://github.com/astefanutti/phantomjs/tree/QTBUG-10094 and was unable to achieve better results :-( |
@adamwong246 would you be able to share a reproducible test case so that I can give a run and confirm or not whether ariya/phantomjs#13243 fixes that issue as well. |
I just tested markdown-pdf with providing |
Please give v7.0.0 of markdown-pdf a go - phantomjs finally fixed this issue |
+1 Thanks a lot! |
I have created simple PDF out of 33 KB .md → 9 MB size... For 8 pages document without pictures it is more than 1 MB/page.
I have checked document in Illustrator and here it is. Text is not stored as text, but as curves. All text. Looks terribly wrong.
For my cases this is a surely not acceptable. Maybe there is an option to include text as text, not as curves?
The text was updated successfully, but these errors were encountered: