Bug 221370 - spell checking should ignore entries in \eqref and \label
Summary: spell checking should ignore entries in \eqref and \label
Status: RESOLVED FIXED
Alias: None
Product: kile
Classification: Applications
Component: general (show other bugs)
Version: 2.0.84
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Michel Ludwig
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-05 12:14 UTC by Christian Trippe
Modified: 2010-01-11 22:52 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:

Attachments
Updated LaTeX syntax file. (36.56 KB, text/xml)
2010-01-07 21:22 UTC, Michel Ludwig 		Details
Updated LaTeX syntax file. (34.78 KB, application/xml)
2010-01-10 13:49 UTC, Michel Ludwig 		Details
Show Obsolete (1) View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Trippe 2010-01-05 12:14:32 UTC 
Version:           2.0.84 (using 4.3.85 (KDE 4.3.85 (KDE 4.4 Beta2)) "release 203", KDE:KDE4:Factory:Desktop / openSUSE_11.2)
Compiler:          gcc
OS:                Linux (i686) release 2.6.31.8-0.1-default

When automatic spell checking is enabled entries in \eqref{...} and \label{..} should be ignored as it is already done for \ref{...}.

And thanks a lot for the On-the-fly spell checking!
Comment 1 Michel Ludwig 2010-01-05 12:30:22 UTC 
Entries under \label shouldn't be spell checked already now.
Comment 2 Christian Trippe 2010-01-05 12:55:41 UTC 
Thanks for the quick response. I am using

rpm -q kile
kile-2.0.84.svn1055245-14.23.i586

with the version also entries in \label are spell checked.
Comment 3 Michel Ludwig 2010-01-05 13:00:39 UTC 
Ok. In revision 1070352 '\eqref' has been added to the commands that contain labels as arguments.

You can update the LaTeX XML file from 'trunk/KDE/kdelibs/kate/syntax/data/latex.xml' manually if you want to.
Comment 4 Christian Trippe 2010-01-06 13:25:19 UTC 
Thanks for the quick fix. \eqref works now nicely with the updated latex.xml file. 

However there is a problem with \label. This works when it is used outside of math environments. However it does not work, i.e. is still spell checked, within math environments such as 'equation', 'align' and 'multline'.

I hope it is okay to reopen the report because of this fact.
Comment 5 Michel Ludwig 2010-01-07 21:22:25 UTC 
Created attachment 39664 [details]
Updated LaTeX syntax file.

Can you try out whether the attached LaTeX syntax file fixes this bug and bug 221383 for you?

Thanks.
Comment 6 Christian Trippe 2010-01-07 21:56:32 UTC 
First, thanks again for your quick response!

> Can you try out whether the attached LaTeX syntax file fixes this bug and bug
> 221383 for you?

bug 221383 is fixed for me with this xml file and also \label is not longer checked in math environment.

However I noticed one regression with your xml file '\footnote' is now spell checked which was not the case before.

I tested that this is fixed with adding 'spellChecking="false"' again to line 460 which was removed.

The last point: I now noticed that the content of '\ref' in a footnote is also spell checked, which is still the case with your updated xml file.
Comment 7 Michel Ludwig 2010-01-10 13:49:59 UTC 
Created attachment 39734 [details]
Updated LaTeX syntax file.

Can you try out this new version? Thanks.
Comment 8 Christian Trippe 2010-01-10 19:29:49 UTC 
The newest version also fixes the problem with footnotes.

Thanks.
Comment 9 Michel Ludwig 2010-01-11 22:52:13 UTC 
Fixed in commit 1073258.