Bug 155280 - provide spell checking correction recommendations in the task editor
Summary: provide spell checking correction recommendations in the task editor
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC All
: P3 enhancement with 1 vote (vote)
Target Milestone: 3.0   Edit
Assignee: Eugene Kuleshov CLA
QA Contact:
URL:
Whiteboard:
Keywords: helpwanted
: 196555 196980 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-08-25 17:05 EDT by Robert Elves CLA
Modified: 2007-08-24 14:11 EDT (History)
4 users (show)

See Also:


Attachments
spellchecking quickfix for comment editor and other editors created trough addTextViewer() (16.71 KB, patch)
2007-08-01 20:31 EDT, Eugene Kuleshov CLA
no flags Details | Diff
mylyn/context/zip (11.90 KB, application/octet-stream)
2007-08-01 20:31 EDT, Eugene Kuleshov CLA
no flags Details
spellchecking quickfix (take 2) (16.10 KB, text/plain)
2007-08-01 23:33 EDT, Eugene Kuleshov CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Robert Elves CLA 2006-08-25 17:05:12 EDT
Eugene suggested that spelling recommendations be made available via content assist.
Another idea would be to populate the context menu with the recommendations.
The first hurdle will be actually getting the recommendations form the spelling engine.
Comment 1 Eugene Kuleshov CLA 2006-08-25 19:11:41 EDT
I think it would be more Eclipse way to have Ctrl-1 or Ctrl-Space kind of content assist. Framework is actually bound to 3.2 API. You may want to look at http://philipptiedt.blogspot.com/2006/06/decorated-fields-in-eclipse.html

Thunderbird is using popup menu for proposals and I still can't get used to it...
Comment 2 Eugene Kuleshov CLA 2006-08-25 20:01:29 EDT
By the way. It should be possible to use context of the current task to create code complation proposals.  I think we talked about this at some point...
Comment 3 Willian Mitsuda CLA 2007-07-14 21:36:07 EDT
*** Bug 196555 has been marked as a duplicate of this bug. ***
Comment 4 Eugene Kuleshov CLA 2007-07-18 12:51:27 EDT
*** Bug 196980 has been marked as a duplicate of this bug. ***
Comment 5 Eugene Kuleshov CLA 2007-08-01 20:31:53 EDT
Created attachment 75179 [details]
spellchecking quickfix for comment editor and other editors created trough addTextViewer()
Comment 6 Eugene Kuleshov CLA 2007-08-01 20:31:54 EDT
Created attachment 75180 [details]
mylyn/context/zip
Comment 7 Eugene Kuleshov CLA 2007-08-01 20:35:34 EDT
This patch is using Eclipse 3.3 API and I had to do some cleanup to make it work.
Comment 8 Eugene Kuleshov CLA 2007-08-01 21:56:00 EDT
Ouch. It seems like I posted it too quick. This patch break hyperlinks for the stack trace elements
Comment 9 Eugene Kuleshov CLA 2007-08-01 23:33:36 EDT
Created attachment 75183 [details]
spellchecking quickfix (take 2)

This patch handle stacktrace hyperlinks properly. 

Apparently I was operoptimistic about hyperlink detectors and we'll need to fix bug 179856 to make it nice. I actually have it fixed here and will submit patch to bug 179856 once this one is resolved.
Comment 10 Robert Elves CLA 2007-08-02 18:42:19 EDT
Nicely done! Patch applied and verified.
Comment 11 Eugene Kuleshov CLA 2007-08-02 18:47:56 EDT
Thanks Rob. Anything else left here or we can resolve it?
Comment 12 Robert Elves CLA 2007-08-02 19:37:10 EDT
Nope I think we're good here, marking resolved...
Comment 13 Martin Oberhuber CLA 2007-08-07 07:25:40 EDT
I'm not currently a Mylyn user, so I apologize if these are silly questions;
bug I stumbled over Eugene's blog and I'd like to get spellcheck fixes for JDT:

 * Will this patch also make spellcheck corrections available in the JDT
   editor (for Javadoc comments)?
 * What download / version / milestone do I need to grab to get the fix into
   my Eclipse 3.3 installation (Mylyn 3.0M1 or what)? What does the target
   milestone 3.0 mean?
 * Why is this bug associated with Component Mylyn : Bugzilla?

Thanks!
Comment 14 Eugene Kuleshov CLA 2007-08-07 09:37:52 EDT
Martin, this bug probably should have been using UI component, since it is related to all Mylyn's task editors. Anyways, it is fixed and you can update Mylyn from the dev update site (its version is between Mylyn 2.0 and 2.1).

Eclipse provides spell checking for all text editors (including JDT's ones) since version 3.2. Version 3.3 includes a dictionary, so you don't have to configure your own one. Though it need to be enabled in Window / Preferences / General / Editors / Text Editors / Spelling and spelling annotations in Window / Preferences / General / Editors / Text Editors / Annotations. As far as I know spell checking for the javadocs is enabled automatically once global spell checking is enabled (and when you have dictionaly).
Comment 15 Martin Oberhuber CLA 2007-08-07 09:53:56 EDT
Thanks Eugene. I've been using the JDT spell checking for a while, and I've come up with a nice user_dict.txt in the meanwhile. But I'm editing this with Emacs outside Eclipse. So what I'd really like is the correction quickfix and the "add to dictionary" quickfix in JDT.

Will I get this when I get Mylyn v 2.0.x from the dev update site?
Comment 16 Eugene Kuleshov CLA 2007-08-07 12:08:04 EDT
Martin, you don't need Mylyn to be able to have spell checking in Java editor. It is supported by JDT out of the box. So, my patch don't have anything to do with that.
Comment 17 Martin Oberhuber CLA 2007-08-08 04:11:25 EDT
Foo! You're right, Ctrl+1 does the spellcheck quickfix for JDT already. Docs have it in "Tips and Tricks (JDT)". 

I guess I didn't find it because I tried ctrl+space only, since no problem marker was visible I didn't get the idea that Ctrl+1 (quickfix) would do it. Also, the text hover over a misspelled word just says "...is not spelled correctly" but gives no hint on how to fix it automatically.

Also, the "Tips and Tricks (JDT)" docs say: "You can make the dictionary also available to the content assist." - but I did not find a way how to do it. I'm wondering whether it would make sense to file a bug asking to make these things more prominently visible for novice users...
Comment 18 Mik Kersten CLA 2007-08-24 14:11:56 EDT
Excellent to see this!  Will consider for New & Noteworthy (added note).