Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 255200 - [Project Sets] Missing "Merge All" in "Import Project Set"
Summary: [Project Sets] Missing "Merge All" in "Import Project Set"
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Team (show other bugs)
Version: 3.4.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.5 M4   Edit
Assignee: Platform Team Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 255201 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-11-13 10:46 EST by Lars Millberg CLA
Modified: 2008-11-14 15:14 EST (History)
1 user (show)

See Also:


Attachments
Adds Merge All (and more) buttons (9.10 KB, patch)
2008-11-13 10:46 EST, Lars Millberg CLA
no flags Details | Diff
Patch v2 (7.42 KB, patch)
2008-11-14 07:54 EST, Lars Millberg CLA
tomasz.zarna: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Millberg CLA 2008-11-13 10:46:12 EST
Created attachment 117785 [details]
Adds Merge All (and more) buttons

I've only recently started to use working sets in the project explorer, and find it quite useful for workspaces of some size.
Our psf files also have the working set stuff, and here comes the issue.
It would be convenient to have an extra button in the "Working Set Exists" dialog that said "Merge All". While we're at it, let's add "Replace All" and "Skip All", too!

I'll attach a patch.

TIA
Comment 1 Tomasz Zarna CLA 2008-11-14 05:14:30 EST
Thanks for the patch Lars. It works fine, but I've got one concern: doesn't the dialog with 7 buttons look a bit too busy to you? What about replacing the do-for-all buttons with a single checkbox saying "Do the same for all working sets from the psf" or simply "Remember my decision" (for current import)? What do you think?
Comment 2 Lars Millberg CLA 2008-11-14 06:16:17 EST
I think an extra check box would work just as fine, as long as it only applies to the current import.

Do you have time to do this, or should I rework the patch?
While we're at it, it would be good to set the accelerator chars, too (the underscores), don't you think?

Comment 3 Lars Millberg CLA 2008-11-14 07:54:46 EST
Created attachment 117896 [details]
Patch v2
Comment 4 Tomasz Zarna CLA 2008-11-14 10:37:17 EST
Perfect! I've released the second patch to HEAD. Thanks Lars!
Comment 5 Lars Millberg CLA 2008-11-14 15:14:27 EST
*** Bug 255201 has been marked as a duplicate of this bug. ***