View Single Post
Old 05-31-2012, 03:01 PM   #329
kiwidude
Calibre Plugins Developer
kiwidude ought to be getting tired of karma fortunes by now.kiwidude ought to be getting tired of karma fortunes by now.kiwidude ought to be getting tired of karma fortunes by now.kiwidude ought to be getting tired of karma fortunes by now.kiwidude ought to be getting tired of karma fortunes by now.kiwidude ought to be getting tired of karma fortunes by now.kiwidude ought to be getting tired of karma fortunes by now.kiwidude ought to be getting tired of karma fortunes by now.kiwidude ought to be getting tired of karma fortunes by now.kiwidude ought to be getting tired of karma fortunes by now.kiwidude ought to be getting tired of karma fortunes by now.
 
Posts: 4,731
Karma: 2197770
Join Date: Oct 2010
Location: Australia
Device: Kindle Oasis
Yeah, there are two things here. The first is the bug in the Remove cover functionality, which is not removing the NCX entry, and also not doing the inline TOC.

The second is whether I should allow a user to select both options in the first place and interrupt the user. Or if behind the scenes I should just detect that the user had both selected and ignore the "Remove existing cover" option in that circumstance, which is probaby my preference.

The issue is that if I allow them to run in sequence (once the bugs are fixed), is that you will by doing so have lost both a link in your internal TOC and in your external TOC to the cover. Now since it was clearly your intent to put a cover into the epub, I see no use case where you intentionally want any existing pointers to the cover being lost?
kiwidude is offline   Reply With Quote