Bug 196867 - [Manifest][Editors] Extensions section tree viewer could use a "Remove" button
Summary: [Manifest][Editors] Extensions section tree viewer could use a "Remove" button
Status: RESOLVED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows XP
: P3 minor (vote)
Target Milestone: 3.4 M2   Edit
Assignee: Peter Friese CLA
QA Contact:
URL:
Whiteboard:
Keywords: bugday, contributed
Depends on:
Blocks:
 
Reported: 2007-07-17 15:40 EDT by Mike Pawlowski CLA
Modified: 2007-09-03 12:47 EDT (History)
2 users (show)

See Also:
mike.pawlowski: review? (wassim.melhem)

Attachments
Patch adds remove button (6.74 KB, patch)
2007-08-31 15:46 EDT, Peter Friese CLA 		no flags Details | Diff
mylyn/context/zip (12.27 KB, application/octet-stream)
2007-08-31 15:47 EDT, Peter Friese CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Pawlowski CLA 2007-07-17 15:40:55 EDT 
Unless vetoed by Wassim.

Add a "Remove" button to
"Extensions" page > "All Extensions" section
under the "Add..." button.

Clicking the remove button (when an object is selected in the tree viewer) will have the same effect as pressing the "Delete" key (when an object is selected in the tree viewer).
Comment 1 Wassim Melhem CLA 2007-07-17 16:12:36 EDT 
We should certainly have a remove button.  a DEL keystroke alone is not sufficient.
Comment 2 Peter Friese CLA 2007-07-27 04:05:16 EDT 
Sounds like a nice bugday candidate. Since I may not change the keywords field, could please someone else flag it?
Comment 3 Chris Aniszczyk CLA 2007-07-27 17:34:23 EDT 
let me know if you want it.
Comment 4 Peter Friese CLA 2007-07-27 17:40:05 EDT 
 (In reply to comment #3)
> let me know if you want it.
Sure.
Comment 5 Chris Aniszczyk CLA 2007-07-27 17:44:07 EDT 
peter, you know you can change your bugzilla id :P?
Comment 6 Peter Friese CLA 2007-08-31 15:46:58 EDT 
Created attachment 77504 [details]
Patch adds remove button

This patch supplies the requested "remove" button. Text is NLSed. I also extracted some evil literals and made them into some nice constants.
Comment 7 Peter Friese CLA 2007-08-31 15:47:02 EDT 
Created attachment 77505 [details]
mylyn/context/zip
Comment 8 Chris Aniszczyk CLA 2007-09-03 12:47:11 EDT 
committed to HEAD. Thanks!