View Single Post
Old 01-03-2020, 03:07 PM   #17
KevinH
Sigil Developer
KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.
 
Posts: 8,899
Karma: 6120478
Join Date: Nov 2009
Device: many
No actually the code already existed in ImportHTML.cpp to handle creating the nav if needed.
I just added the code to create a default ncx and somehow ended up with two entries in the manifest for the same file.

I just pushed fixes for that. So we should now be okay (no crashing anymore).

That said the default NCX we create has a bad reference to Start Text/Section1.xhtml but that goes away when you Create the Table Of Contents.

It will take some diggng to find out the name and location of the first xhtml imported and to fixup the NCX code so it is always correct. I will look into adding that last piece over the weekend sometime.

None of this matters for AddExisting...
KevinH is online now   Reply With Quote