Bug 252629 - Make spell checking of comments configurable
Summary: Make spell checking of comments configurable
Status: RESOLVED FIXED
Alias: None
Product: kate
Classification: Applications
Component: part (show other bugs)
Version: SVN
Platform: Ubuntu Linux
: NOR wishlist
Target Milestone: ---
Assignee: KWrite Developers
URL:
Keywords: investigated, triaged
Depends on:
Blocks:
 
Reported: 2010-09-28 11:12 UTC by Thomas Damgaard
Modified: 2018-09-23 17:31 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Damgaard 2010-09-28 11:12:33 UTC
Version:           2.0.85 (using Devel) 
OS:                Linux

Kile is a LaTeX editor but its spellchecker does not understand TeX source code.
Example:
``This is a quote''

In this example the text `` is marked as a spelling error. However, `` and '' are used when quoting text in TeX. Thus, it should not be seen as an error.

It seems that the spellcheck is already somewhat context aware because it does not check words starting with '\'. 

It would probably be relevant to include a functionality to ignore comments. It is often not desirable to spellcheck comments.

Reproducible: Always
Comment 1 Michel Ludwig 2010-09-28 11:26:33 UTC
Please update to KDE 4.5.1. The spell checking problems that you mention have been fixed there.

Also, please ensure that you don't have an old latex.xml lying around (in ~/.kde/share/apps/katepart/syntax/). Thanks.

Regarding the spell checking of comments, some users like it, others don't. So, I think the easiest would be to make this more configurable.
Comment 2 Dominik Haumann 2010-11-07 21:13:39 UTC
@Thomas: As Michel said, please try again with a more recent Kate version (or Kile that uses a newer version of Kate). Can we close this report?
Comment 3 Thomas Damgaard 2010-11-08 09:55:52 UTC
I will try it soon.
Comment 4 Dominik Haumann 2011-08-12 11:53:39 UTC
Any news?
Comment 5 Thomas Damgaard 2011-08-12 19:58:25 UTC
I will try as soon as possible.
Comment 6 Thomas Damgaard 2011-08-14 17:51:21 UTC
> Also, please ensure that you don't have an old latex.xml lying around (in
~/.kde/share/apps/katepart/syntax/). Thanks.

ls: cannot access /home/tdn/.kde/share/apps/katepart/syntax/: No such
file or directory
Comment 7 Thomas Damgaard 2011-08-14 17:53:06 UTC
It seems to have been fixed in current version of kile.
Comment 8 Dominik Haumann 2011-08-14 19:39:51 UTC
Then maybe it's .kde4 instead of .kde.
Thanks for the feedback.
Comment 9 Thomas Damgaard 2011-08-14 20:16:44 UTC
> --- Comment #8 from Dominik Haumann <dhdev gmx de>  2011-08-14 19:39:51 ---
> Then maybe it's .kde4 instead of .kde.

I do not have .kde4. I have ~/.kde/share/apps/katepart/ but it only contains:
total 16K
-rw-rw-r-- 1 tdn tdn 4,4K 2011-01-30 20:44 katepartsimpleui.rc
-rw-rw-r-- 1 tdn tdn 6,4K 2011-01-30 22:31 katepartui.rc
Comment 10 Andrew Crouthamel 2018-09-22 02:08:37 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least 15 days. Please provide the requested information as soon as possible and set the bug status as REPORTED. Due to regular bug tracker maintenance, if the bug is still in NEEDSINFO status with no change in 30 days, the bug will be closed as RESOLVED > WORKSFORME due to lack of needed information.

For more information about our bug triaging procedures please read the wiki located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please set the bug status as REPORTED so that the KDE team knows that the bug is ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!
Comment 11 Dominik Haumann 2018-09-23 17:31:11 UTC
I think this should work now with the new spell check support in the xml files including the encoding mapping.