[Enhancement] Spell check should cover nav.xhtml

Bug #1942773 reported by Martin Persson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
calibre
Fix Released
Undecided
Unassigned

Bug Description

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.

Revision history for this message
Martin Persson (martinper) wrote :

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!

Revision history for this message
Kovid Goyal (kovid) wrote : Re: calibre bug 1942773

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.

Revision history for this message
Martin Persson (martinper) wrote :

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.

Revision history for this message
Kovid Goyal (kovid) wrote : Fixed in master

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

 status fixreleased

Changed in calibre:
status: New → Fix Released
Revision history for this message
Martin Persson (martinper) wrote :

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

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers