Bug 221370

Summary: spell checking should ignore entries in \eqref and \label
Product: [Applications] kile Reporter: Christian Trippe <christiandehne>
Component: generalAssignee: Michel Ludwig <michel.ludwig>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: 2.0.84   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: Updated LaTeX syntax file.
Updated LaTeX syntax file.

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.