Bug 522096 - [Project Explorer] "Close all included projects" on working set
Summary: [Project Explorer] "Close all included projects" on working set
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.7   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 4.8 M4   Edit
Assignee: Lucas Bullen CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy, usability
Depends on:
Blocks: 528035
  Show dependency tree
 
Reported: 2017-09-10 10:32 EDT by Mickael Istria CLA
Modified: 2018-07-26 15:23 EDT (History)
3 users (show)

See Also:


Attachments
Open and close projects contained in a working set (62.87 KB, image/png)
2017-09-10 10:43 EDT, Conrad Groth CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mickael Istria CLA 2017-09-10 10:32:00 EDT
To better handle some use-cases with working sets, it would be very convenient to provide a "Close all included projects" action on working set(s). It's also important that this action work also in case selection is multiple working sets.
Comment 1 Conrad Groth CLA 2017-09-10 10:43:47 EDT
Created attachment 270138 [details]
Open and close projects contained in a working set
Comment 2 Conrad Groth CLA 2017-09-10 10:44:48 EDT
(In reply to Conrad Groth from comment #1)
> Created attachment 270138 [details]
> Open and close projects contained in a working set

The attachment shows the context menu of a working set in the package explorer. It would be nice to have that feature also in the project explorer.
Comment 3 Dani Megert CLA 2017-11-07 10:25:50 EST
The naming of the action (in the Package Explorer) is wrong, see bug 522096.
Comment 4 Conrad Groth CLA 2017-11-12 15:35:26 EST
(In reply to Dani Megert from comment #3)
> The naming of the action (in the Package Explorer) is wrong, see bug 522096.

You referenced this bug?
Comment 5 Dani Megert CLA 2017-11-13 04:48:56 EST
(In reply to Conrad Groth from comment #4)
> (In reply to Dani Megert from comment #3)
> > The naming of the action (in the Package Explorer) is wrong, see bug 522096.
> 
> You referenced this bug?

Thanks for noticing. It's bug 526946.
Comment 6 Eclipse Genie CLA 2017-11-22 13:07:42 EST
New Gerrit change created: https://git.eclipse.org/r/112098
Comment 7 Mickael Istria CLA 2017-11-27 16:53:18 EST
Thanks Lucas for the patch and Conrad for the review!
Let's keep this bug open to track the addition to N&N.
Comment 9 Eclipse Genie CLA 2017-11-27 17:37:01 EST
New Gerrit change created: https://git.eclipse.org/r/112392
Comment 11 Alexander Kurtakov CLA 2017-11-28 08:22:57 EST
N&N applied.
Comment 12 Dani Megert CLA 2017-11-28 09:02:43 EST
This doesn't work for me on 'Other Projects' working set.
Comment 13 Lucas Bullen CLA 2017-11-30 15:39:40 EST
(In reply to Dani Megert from comment #12)
> This doesn't work for me on 'Other Projects' working set.

I think this is actually a larger issue and should be addressed in a different bug. The "Other Projects" Working Set isn't a WS, it is actually just an empty object that is referenced, so does not set off any of the "is this a IWorkingSet" tests.
Comment 14 Mickael Istria CLA 2017-11-30 16:38:15 EST
(In reply to Lucas Bullen from comment #13)
> I think this is actually a larger issue and should be addressed in a
> different bug. The "Other Projects" Working Set isn't a WS, it is actually
> just an empty object that is referenced, so does not set off any of the "is
> this a IWorkingSet" tests.

Could the test easily be changed into "can adapt to an array of projects", and to have adapters to turn an IWorkingSet and the OtherProjects thing into such arrays of project?
That said, I support the idea of a separate bug.
Comment 15 Dani Megert CLA 2017-12-01 10:17:49 EST
(In reply to Mickael Istria from comment #14)> That said, I support the idea of a separate bug.

Yeah, it's easy to provide a half-finished fix and then outsource the compete one. Do whatever you see fit. I don't consider this bug to be fixed though.
Comment 16 Mickael Istria CLA 2017-12-01 13:04:35 EST
(In reply to Dani Megert from comment #15)
> Yeah, it's easy to provide a half-finished fix and then outsource the
> compete one. Do whatever you see fit. I don't consider this bug to be fixed
> though.

Iterative development FTW!
Bugs with scope that goes increasing and increasing are often dis-motivating and make tracking progress and value delivery harder. Iterative scales better and allows to re-prioritize things with more agility. That's the idea of creating another bug: we consider a big part of the value as properly shipped, and create a new issue to consider some additional value for later, when it becomes top-priority for a contributor.
This "Half-finished work" already provides all the value I expected to be produced when I reported the bug, and I believe it also covers Conrad's use case. The patch answering the initial request is I believe a proper reason to mark this bug as RESOLVED ;)

Bug 528035 new tracks the missing bits you've identified.
Comment 17 Lucas Bullen CLA 2018-01-22 12:59:10 EST
Verified in I20180122-0800