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...
|