|01-09-2009, 11:32 PM||#1|
Join Date: Jan 2009
nytimes, missing images in EPUB
The nytimes recipe works great, but images don't show up when using the EPUB format (on the PRS-505 device itself).
I think the reason is because mobile.nytimes.com names their JPEG images with .cr extensions, and the Reader doesn't approve. As an experiment, if I:
1) save the intermediate HTML (after calling feeds2epub),
2) rename all images to have .jpg extensions,
3) edit the corresponding HTML to refer to the new image filenames,
4) convert the HTML to EPUB
...then the PRS-505 displays images just fine.
I haven't studied Calibre's codebase too deeply yet, so I'll ask: what is the best approach to fixing this? Would it be to edit the nytimes recipe to rename all images appropriately?
|01-10-2009, 01:52 PM||#2|
Creator of calibre
Join Date: Oct 2006
Location: Mumbai, India
It's a good idea to fix this for all recipe downloads, since it's probably hapening for others as well. I would suggest making this change in the process_images method in the web.fetch.simple module. The best way to do it would be to read in the image using QImage and write it out as jpeg after checking that the image is valid. If invalid, then dont write the image. Go ahead and submit a patch or open a ticket and I'll fix it when I get some time.
|Thread Tools||Search this Thread|
|Thread||Thread Starter||Forum||Replies||Last Post|
|ePub creation tools : what's missing ? wishlist / dialogue||zelda_pinwheel||ePub||103||02-08-2013 05:36 AM|
|epub/mobi --> RTF: italics missing||acanthicus||Calibre||0||06-09-2010 05:20 PM|
|Where are epub images stored?||janneman||Sigil||3||01-06-2010 05:43 AM|
|Missing images||Littermate||Sigil||3||12-18-2009 03:03 PM|
|Help with images in EPUB conversions, please||jackie_w||Calibre||11||10-30-2009 04:29 PM|