Bug 47065 - Execute spell-checking only on non-code parts
Summary: Execute spell-checking only on non-code parts
Status: RESOLVED INTENTIONAL
Alias: None
Product: kdevelop
Classification: Applications
Component: general (show other bugs)
Version: 4.0.0
Platform: unspecified Other
: NOR wishlist
Target Milestone: ---
Assignee: KDevelop-Devel List
URL:
Keywords:
: 61282 80865 128713 (view as bug list)
Depends on:
Blocks:
 
Reported: 2002-08-27 16:03 UTC by greendeath
Modified: 2010-09-23 17:55 UTC (History)
6 users (show)

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 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 :)