Bug 7945 - Documentation has spelling and grammar errors
Summary: Documentation has spelling and grammar errors
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Documentation (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.18.0
Assignee: William Sjöblom
URL:
Keywords: prosaic, tobfa_tester
Depends on:
Blocks:
 
Reported: 2022-06-10 15:51 CEST by Pierre Ossman
Modified: 2024-10-29 16:59 CET (History)
3 users (show)

See Also:
Acceptance Criteria:


Attachments

Description Pierre Ossman cendio 2022-06-10 15:51:14 CEST
We don't do any spell checks on the product documentation, which means that errors do creep in now and then. This lowers the overall impression of not just the documentation, but the product as a whole. So we would like to remedy this and have a proper spell check of everything.

There is an extension for Sphinx that does spell checking using Enchant:

https://sphinxcontrib-spelling.readthedocs.io/en/latest/

However, these days there are also grammar checking tools, such as LanguageTool¹.  So we might want to pass the data through such a tool to get both the grammar and spelling checked.

¹ https://languagetool.org
Comment 1 Pierre Ossman cendio 2022-06-10 15:51:55 CEST
This should also make it easier to enforce British vs American spelling (bug 5561).
Comment 2 Pierre Ossman cendio 2022-06-10 15:57:01 CEST
(In reply to Pierre Ossman from comment #0)
> There is an extension for Sphinx that does spell checking using Enchant:
> 
> https://sphinxcontrib-spelling.readthedocs.io/en/latest/

A quick run of this gives 737 hits. Many things are false positives (like product names), but there are also many actual problems found.
Comment 3 Pierre Ossman cendio 2024-01-10 14:01:12 CET
There is this language server (and extension for vscode) that can be used for reStructuredText:

https://valentjn.github.io/ltex/

It supports using our LanguageTool Premium accounts for better checks.

It requires another language server to parse reStructuredText files, though, which is crashing on some of our files:

https://github.com/swyddfa/esbonio/issues/714


The above doesn't give us automatic checks, but it can at least help whilst actively working on the documentation.
Comment 4 William Sjöblom cendio 2024-06-17 14:03:35 CEST
FWIW, another option is just copying and pasting portions of the rendered HTML into LanguageTools web spell checker. As it handles rich text, it works really well.
Comment 12 William Sjöblom cendio 2024-10-04 15:06:39 CEST
The tag has now been spell and grammar checked using LanguageTool.
Comment 13 Tobias cendio 2024-10-08 12:33:00 CEST
Went through all the grammar and spelling corrections of the documentation. Generally it looks very good, with two exceptions:

* client_macos.rst: singular AltGr followed by 'were'

* tl-config.1.rst: incorrect indentation under --all-entries option

Reopening to get this fixed.
Comment 15 Tobias cendio 2024-10-08 12:40:44 CEST
Issues mentioned in comment 13 have been resolved.

Closing.

Note You need to log in before you can comment on or make changes to this bug.