add missing options in confix dialog
update + cleanup document variables section
Details
Details
- Reviewers
dhaumann - Group Reviewers
Kate Documentation - Commits
- R40:1f0ce03670d1: Update configuring.docbook to ktexteditor 5.38
checkXML5 index.docbook
Diff Detail
Diff Detail
- Repository
- R40 Kate
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Thanks for working on this. The patch looks pretty good, and after another iteration I'm confident we can merge this.
| doc/katepart/configuring.docbook | ||
|---|---|---|
| 168 | Typo: hoover -> hover | |
| 1295 | This will get outdated again and again. Maybe just say: The variables listed here documents the current state. More ... Or similar? | |
| 1313 | spell-checking in the auto-brackets section looks wrong. Could you have a second look? | |
| 1389 | remove "on" | |
| 1474 | Instead of the term 'conversion' we may want to use "expansion". But I may be wrong, since I am not a native English speaker. | |
| doc/katepart/development.docbook | ||
| 276 | Good catch! :-) | |
| doc/katepart/configuring.docbook | ||
|---|---|---|
| 1295 | I'd prefer to have a date/version similar to date/releaseinfo in the docbook header. | |
| 1389 | frameworks/ktexteditor/src/variableeditor/variablelineedit.cpp:200 has this typo as well: | |
| doc/katepart/configuring.docbook | ||
|---|---|---|
| 1389 | Fixed typo in variablelineedit.cpp with https://commits.kde.org/ktexteditor/fd670c9d44ff1923266f2547b88b164cd13e6d20 | |