Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 287899

Summary: consume CheckBoxTreeDialog for KeywordsDialog
Product: z_Archived Reporter: Shawn Minto <shawn.minto>
Component: MylynAssignee: Frank Becker <eclipse>
Status: RESOLVED FIXED QA Contact:
Severity: trivial    
Priority: P2 CC: eclipse, robert.elves
Version: unspecified   
Target Milestone: 3.4   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on:    
Bug Blocks: 290614    
Attachments:
Description Flags
commited patch
none
mylyn/context/zip none

Description Shawn Minto CLA 2009-08-27 18:05:41 EDT 
We now have a CheckBoxTreeDialog that is in the o.e.m.commons.ui plugin as per bug 279334.  The KeywordsDialog should use this so we do not have duplication of this code or we should eliminate the KeywordsDialog all together.
Comment 1 Steffen Pingel CLA 2009-08-27 18:51:40 EDT 
Frank, would you be interested in refactoring this to get rid of the code duplication?
Comment 2 Frank Becker CLA 2010-02-25 13:42:04 EST 
(In reply to comment #1)
> Frank, would you be interested in refactoring this to get rid of the code
> duplication?

Steffen,

should I now start with this?
Comment 3 Steffen Pingel CLA 2010-02-25 18:05:19 EST 
If you time it would be great if you could work on this.
Comment 4 Frank Becker CLA 2010-02-28 15:54:01 EST 
Created attachment 160432 [details]
commited patch

I eliminate the KeywordsDialog!
Comment 5 Frank Becker CLA 2010-02-28 15:54:04 EST 
Created attachment 160433 [details]
mylyn/context/zip
Comment 6 Frank Becker CLA 2010-02-28 15:55:52 EST 
I think we are done here!