Bug 47065

Summary: Execute spell-checking only on non-code parts
Product: [Applications] kdevelop Reporter: greendeath
Component: generalAssignee: KDevelop-Devel List <kdevelop-devel>
Status: RESOLVED INTENTIONAL    
Severity: wishlist CC: ackdrummaster, domseichter, lashack, roger.larsson, shafff, uetsah
Priority: NOR    
Version: 4.0.0   
Target Milestone: ---   
Platform: unspecified   
OS: Other   
Latest Commit: Version Fixed In:

Description greendeath 2002-08-27 15:53:28 UTC
(*** This bug was imported into bugs.kde.org ***)

Package:           kdevelop
Version:           KDE 3.0.7 
Severity:          wishlist
Installed from:    Unspecified
Compiler:          Not Specified
OS:                Not Specified
OS/Compiler notes: Not Specified

Implement Spell Check for i18n() calls arguments as menu command (in foreground) or/and in background thread. And mark misspelled words with color/underscoring. Like OpenOffice/MS Word...

All translators and proofreaders especially will benefit from this feature.

(Submitted via bugs.kde.org)
Comment 1 Amilcar do Carmo Lucas 2003-07-15 17:02:06 UTC
*** Bug 61282 has been marked as a duplicate of this bug. ***
Comment 2 Andreas Pakulat 2007-02-23 12:12:42 UTC
KDevelop uses kate as editor which supports spell checking of files.
Comment 3 Matt Rogers 2007-02-23 19:47:14 UTC
But they want spell checking of the strings in the i18n calls, which may or may not be supported by kate. With the new text editing stuff, duchain highlighting, etc. We should have the ability to do this for KDevelop 4. Reopening.
Comment 4 Andreas Pakulat 2008-06-29 15:17:10 UTC
First thing to do is provide a KTE interface for spellchecking. Changing the title to be more generic
Comment 5 Andreas Pakulat 2008-06-30 23:11:03 UTC
*** Bug 128713 has been marked as a duplicate of this bug. ***
Comment 6 Vito De Tullio 2008-07-01 12:09:24 UTC
*** This bug has been confirmed by popular vote. ***
Comment 7 uetsah 2009-12-12 12:47:08 UTC
If this is ever implemented, then - in addition to strings in i18n calls - spellchecking should also apply to doxygen comments (or maybe even comments in general).
Comment 8 ackdrummaster 2010-01-04 10:26:15 UTC
I'm not the best programmer in the world here so I cant help much coding wise, however would it be possible to have the spell check find the "" marks or the // one or */ /* and only check those things with in that? tell me what you think. 

thanks
Comment 9 Milian Wolff 2010-01-07 13:17:22 UTC
Fixed/Implemented in KDE 4.4 - no? Please try it out and open up designated bug reports for the glitches you find there.
Comment 10 Nick Shaforostoff 2010-05-04 20:19:48 UTC
i dare to reopen this report. KDevelop 0.10.2 still doesn't skip code parts during Tools -> Spellcheck.

I tried enabling extensions in Tools -> Configure editor, but nothing changed.

I'd like to check i18n'd strings and (optionally) comments
Comment 11 Andreas Pakulat 2010-05-04 20:38:52 UTC
(In reply to comment #9)
> Fixed/Implemented in KDE 4.4 - no? Please try it out and open up designated bug
> reports for the glitches you find there.

How is this fixed for KDE 4.4? I can't see a KTE interface to execute spell-checking on some predefined string. Let alone the part using duchain to find out the strings of i18n calls.
Comment 12 Milian Wolff 2010-07-09 21:10:50 UTC
But by default only string contents and comments are spellchecked. i18n() contains strings, so everything should be fine? Why would we need kde-special handling for i18n?

Please convince me that this is still needed, otherwise I'll just close it.
Comment 13 Roger Larsson 2010-07-09 22:33:24 UTC
*** Bug 80865 has been marked as a duplicate of this bug. ***
Comment 14 Milian Wolff 2010-09-23 17:47:01 UTC
I just talked to Andreas: His wish was for example this:

i18n("please spell check this string");
QFile("/dont/spellcheck/this.txt");

It's simply not possible to implement this, as there is no /useful/ way to implement this. In the end a user would have to decide whether arguments to a function should get spell checked or not. Far too much effort for imo not much gain. Closing finally, as the main part (i.e. only string contents and comments get spell checked) is implemented in Kate.
Comment 15 Andreas Pakulat 2010-09-23 17:55:12 UTC
(In reply to comment #14)
> I just talked to Andreas: His wish was for example this:
> 
> i18n("please spell check this string");
> QFile("/dont/spellcheck/this.txt");

Minor nitpick: It was not my wish, that was simply how I understood the report
taking into account that kate already checks only comments and strings and yet
the report was re-opened. I don't personally use spell-checking at all, hence
have no wishes in any way related to that :)