Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 274393 - [target] Move... button on pref page enabled for multi-selection
Summary: [target] Move... button on pref page enabled for multi-selection
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.5   Edit
Hardware: PC Linux
: P3 trivial (vote)
Target Milestone: 3.5 RC1   Edit
Assignee: Chris Aniszczyk CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-04-29 15:37 EDT by Curtis Windatt CLA
Modified: 2009-05-04 13:08 EDT (History)
1 user (show)

See Also:
caniszczyk: review+
curtis.windatt.public: review+

Attachments
Patch (1.89 KB, patch)
2009-04-30 14:06 EDT, Ankur Sharma CLA 		caniszczyk: iplog+
Details | Diff
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 Curtis Windatt CLA 2009-04-29 15:37:35 EDT 
Target platform pref page
Select multiple targets
Hit the move... button
Choose a destination
= Only one of the targets gets moved

Since pressing the button only affects the first selected definition, it should only be enabled for single selections.  The same as the edit button.
Comment 1 Curtis Windatt CLA 2009-04-29 15:38:21 EDT 
This can wait until RC1.
Comment 2 Ankur Sharma CLA 2009-04-30 14:06:38 EDT 
Created attachment 133985 [details]
Patch

Added check for the selection size too as done for the Edit buttonalready
Comment 3 Chris Aniszczyk CLA 2009-05-03 18:43:38 EDT 
done.

> 20090503

Thanks for the contribution Ankur!
Comment 4 Curtis Windatt CLA 2009-05-04 10:43:34 EDT 
RC1 requires review by two committers.  I will review.
Comment 5 Curtis Windatt CLA 2009-05-04 13:08:00 EDT 
+1 Verified.