[Enhancement] Spell check should cover nav.xhtml

[Martin Persson]

When using the spell checker in calibre editor, it seems it only checks regular content files in the EPUB (3) fileset, i.e. it excludes nav.xhtml from being checked.

It would be great if spell checking in calibre cover nav.xhtml only. Nav.xhtml could be included in the spine of the book, i.e. part of the regular "book content". But even if not included in the spine, typos in nav.xhtml will have effects on the end user, for whom the navigation file will probably be used and rendered in the reading system.

Tested on Windows, calibre editor v.5.26. Apologies if the feature request is not valid, but couldn't find any previously reported issue on this topic.

[Martin Persson (martinper)]

Hi again. This should perhaps be considered a bug, if the user expects nav.xhtml to be checked and it is not. Feel free to remove the feature request character on this issue if you agree :-)

Thanks!

[Kovid Goyal (kovid)]

If it is included in the spine it will be checked. If it is not
included, then I believe it wont be, though I could be
misremembering.

This is true of all HTML files, not nav.html in particular.

[Martin Persson (martinper)]

I see. I think this would be a valuable fix. One example of this creating odd results is the fixing of a typo in a content document heading with the spell checker, but still having the typo remaining in nav.xhtml, causing a mismatch between navigation document heading and the corresponding target location in the content file.

[Kovid Goyal (kovid)]

Fixed in branch master. The fix will be in the next release. calibre is usually released every alternate Friday.

 status fixreleased

[Martin Persson (martinper)]

Awesome! Thank you very much Kovid for such a prompt fix!