medium-converter icon indicating copy to clipboard operation
medium-converter copied to clipboard

Images are added to the article in a very misshaped size

Open vongohren opened this issue 6 years ago • 6 comments

Feks: https://media.consensys.net/blockchain-vs-distributed-ledger-technologies-1e0289a87b16

The pdf of this one becomse like the following pdf. With huge pictures which cuts of over half of the picture itself. media.consensys.net-blockchain-vs-distributed-ledger-technologies.pdf

vongohren avatar Aug 12 '18 17:08 vongohren

I don't think it depends of this project but of Docverter, since this project does nothing more than retrieving Markdown from Medium and passing it to docverter, which manage the layout.

But since @DanielArthurUK is working on an image-related issue with Medium-parser (on which is also based this project), maybe you can take at look at his messages here.

Finally, I've forked this project to make it work with MarkdownToPdf instead of Docverter, maybe you could see if it resolves your problem as a temporary solution ? https://github.com/Clovis-Portron/medium-converter

Cl0v1s avatar Aug 14 '18 16:08 Cl0v1s

@Clovis-Portron well i went here: https://clovis-portron.github.io/medium-converter/ And from there i got a PDF on the same medium article, with no images. So I dont know what is best, to big or none :P

vongohren avatar Aug 14 '18 19:08 vongohren

So it's definitely an issue with medium-parser, so let's wait for @DanielArthurUK's magic. :D

Cl0v1s avatar Aug 16 '18 09:08 Cl0v1s

Yes this is an issue I highlighted with medium-parser, see gunar/medium-parser#13

I will be opening a PR to fix this in the coming days, but it is likely this fix will come in the form of a major version to fix other issues that currently exist with this library. The major version will change the way the library is used so you may have to change some of your existing code if you wish to upgrade.

Also, thank you for believing in me @Clovis-Portron 😂 I am not sure if it will be magic but hopefully it will work ✨

danarth avatar Aug 16 '18 14:08 danarth

@Clovis-Portron just an update on this issue - I opened a PR last week for medium-parser, just waiting for @gunar to review and hopefully merge 🤞 Once that's done I'll open a PR here to update medium-converter to the new version. There are a bunch of other features too, feel free to check out my PR: gunar/medium-parser#15

danarth avatar Aug 23 '18 11:08 danarth

Great job! I will take a look :D

Cl0v1s avatar Aug 24 '18 10:08 Cl0v1s