![]() |
#1321 | |
null operator (he/him)
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 21,725
Karma: 29711016
Join Date: Mar 2012
Location: Sydney Australia
Device: none
|
Quote:
BR |
|
![]() |
![]() |
![]() |
#1322 |
Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 1,760
Karma: 9918418
Join Date: Feb 2013
Location: Here on the perimeter, there are no stars
Device: Kobo H2O, iPad mini 3, Kindle Touch
|
Okay, I’ve contacted Kovid to request that he update the first post of this thread with RC2. (kiwidude hasn’t connected in a couple of weeks.)
|
![]() |
![]() |
Advert | |
|
![]() |
#1323 |
Bibliophagist
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 46,212
Karma: 168983734
Join Date: Jul 2010
Location: Vancouver
Device: Kobo Sage, Libra Colour, Lenovo M8 FHD, Paperwhite 4, Tolino epos
|
That was a truly weird glitch. The modify epub saved settings in gui.json ( the lines between "modify epub plugin:options dialog:settings": { and }, ) had a couple of lines in the middle which looked like someone had banged their head on the keyboard. After deleting that section, Modify Epub came up with nothing selected. After setting my usual settings, everything seemed to be working again and that section in gui.json was recreated.
Weird and wonderful toys, these computers. |
![]() |
![]() |
![]() |
#1324 |
Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 1,760
Karma: 9918418
Join Date: Feb 2013
Location: Here on the perimeter, there are no stars
Device: Kobo H2O, iPad mini 3, Kindle Touch
|
v1.4.0 Released
I have been informed that v1.4.0 has been attached to the first post, making it the new official release version.
|
![]() |
![]() |
![]() |
#1325 |
Resident Curmudgeon
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 79,757
Karma: 145864619
Join Date: Nov 2006
Location: Roslindale, Massachusetts
Device: Kobo Libra 2, Kobo Aura H2O, PRS-650, PRS-T1, nook STR, PW3
|
|
![]() |
![]() |
Advert | |
|
![]() |
#1326 |
Resident Curmudgeon
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 79,757
Karma: 145864619
Join Date: Nov 2006
Location: Roslindale, Massachusetts
Device: Kobo Libra 2, Kobo Aura H2O, PRS-650, PRS-T1, nook STR, PW3
|
I'm seeing more and more eBooks that are specifying images in CSS. Because of this, Modify ePub gets it wrong when looking for unused images.
If images are going to be deleted as unused because of being specified in the CSS, Modify ePub should be looking at the CSS for images as part of the function. Can this please be fixed so it does look at CSS as well as HTML to see if any images are unused? Thanks. |
![]() |
![]() |
![]() |
#1327 |
Well trained by Cats
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 31,059
Karma: 60358908
Join Date: Aug 2009
Location: The Central Coast of California
Device: Kobo Libra2,Kobo Aura2v1, K4NT(Fixed: New Bat.), Galaxy Tab A
|
Is there a reason that it CAN'T look in the CSS?
![]() Just a bit confused, since Modify can clean the CSS of unused entries. I can understand that it may be tricky, as these pesky book designers find all the different ways to do something. But I would expect the searching for the images Pathname should succeed, even if it did not tell you HOW (the code worked). |
![]() |
![]() |
![]() |
#1328 | |
Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 1,760
Karma: 9918418
Join Date: Feb 2013
Location: Here on the perimeter, there are no stars
Device: Kobo H2O, iPad mini 3, Kindle Touch
|
Yes, and I go into that over here.
Quote:
|
|
![]() |
![]() |
![]() |
#1329 |
Resident Curmudgeon
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 79,757
Karma: 145864619
Join Date: Nov 2006
Location: Roslindale, Massachusetts
Device: Kobo Libra 2, Kobo Aura H2O, PRS-650, PRS-T1, nook STR, PW3
|
The problem is that if Modify ePub cannot look in SS for used images, then the option to remove unused images really should go as it's broken in the current form. It can and does delete images that are being used.
So there's no way to parse the CSS for just images? |
![]() |
![]() |
![]() |
#1330 | |
Well trained by Cats
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 31,059
Karma: 60358908
Join Date: Aug 2009
Location: The Central Coast of California
Device: Kobo Libra2,Kobo Aura2v1, K4NT(Fixed: New Bat.), Galaxy Tab A
|
Quote:
For each name in Images section (of the files report), test for the name (foo.jpg, bar.png...) we don't care HOW it is used in the CSS, so it is just a T/FTrue = skip the clean, error on the safe side. It may be specified, but never ever rendered. Cleaning the CSS first might raise the effectivenss Last edited by theducks; 10-26-2019 at 08:15 PM. Reason: clarify reasoning |
|
![]() |
![]() |
![]() |
#1331 |
Well trained by Cats
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 31,059
Karma: 60358908
Join Date: Aug 2009
Location: The Central Coast of California
Device: Kobo Libra2,Kobo Aura2v1, K4NT(Fixed: New Bat.), Galaxy Tab A
|
![]() My eyes glazed over. That is not simple. Apologies ![]() BTW Does Reports (part of the editor), even inventory the CSS for file call out? |
![]() |
![]() |
![]() |
#1332 | ||
Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 1,760
Karma: 9918418
Join Date: Feb 2013
Location: Here on the perimeter, there are no stars
Device: Kobo H2O, iPad mini 3, Kindle Touch
|
Quote:
Quote:
|
||
![]() |
![]() |
![]() |
#1333 | |
Well trained by Cats
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 31,059
Karma: 60358908
Join Date: Aug 2009
Location: The Central Coast of California
Device: Kobo Libra2,Kobo Aura2v1, K4NT(Fixed: New Bat.), Galaxy Tab A
|
Quote:
That is the where/how many used Sigil has a similar report One of the items: Images gives a count of uses (and the summary of the size and type ![]() |
|
![]() |
![]() |
![]() |
#1334 |
Resident Curmudgeon
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 79,757
Karma: 145864619
Join Date: Nov 2006
Location: Roslindale, Massachusetts
Device: Kobo Libra 2, Kobo Aura H2O, PRS-650, PRS-T1, nook STR, PW3
|
Can anyone who knows Python and Calibre plugins please have a go at fixing Modify ePub so it check the CSS for image links before deleting images that are linked in CSS only?
|
![]() |
![]() |
![]() |
#1335 |
Not Quite Dead
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 195
Karma: 654170
Join Date: Jul 2015
Device: Paperwhite 4; Galaxy Tab
|
All: thanks for the Modify ePub plug-in update. Every file that comes into my Calibre library is subjected to its cleansing rays.
On the issue of the fail on deleting images referenced from the styles: Yes, kind of awkward since an image can be deleted but the actual style rule referencing it remains in the CSS. Kind of a smoking gun... For me, it is not much of an issue...yet. I have not seen any books other than my own which utilize the basic sanitary practice of placing heading/dropcap/scenebreak images from the CSS. I keep that "remove unused images" unchecked in the plug-in options for that reason — and also because it is not uncommon to find a book in which image links (esp. maps in history books) are cruelly orphaned, and I want a chance to correct that. However, it would be very nice to have the plug-in be able to account for these style references when I am using it (sometimes) to blast away 100 files because some...person who needs instruction...has placed that many scenebreak images in the book, carefully copying and renaming the image in each instance. I used to delete those files one at a time but the plug-in makes that easy — as long as I keep aware of the CSS fail. The Calibre Check tool finds the CSS-referenced image fine. The Calibre Reports tool also finds the CSS-referenced image fine. If only there were a way for the knowledge in either of those tools to be known by Modify ePub... Last edited by Brett Merkey; 11-01-2019 at 02:48 PM. |
![]() |
![]() |
![]() |
Tags |
modify epub |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
[GUI Plugin] Quality Check | kiwidude | Plugins | 1251 | 07-07-2025 09:13 PM |
[GUI Plugin] Open With | kiwidude | Plugins | 404 | 02-21-2025 05:42 AM |
[GUI Plugin] Manage Series | kiwidude | Plugins | 167 | 07-28-2024 03:07 PM |
Modify ePub plugin dev thread | kiwidude | Development | 346 | 09-02-2013 05:14 PM |
[GUI Plugin] Plugin Updater **Deprecated** | kiwidude | Plugins | 159 | 06-19-2011 12:27 PM |