Bug 157878 - [CommonNavigator] Confusing working set menu items when showing working sets at top level
Summary: [CommonNavigator] Confusing working set menu items when showing working sets ...
Status: RESOLVED WORKSFORME
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.3   Edit
Hardware: All All
: P2 normal (vote)
Target Milestone: ---   Edit
Assignee: Francis Upton IV CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords: usability
Depends on:
Blocks:
 
Reported: 2006-09-19 14:13 EDT by Kim Horne CLA
Modified: 2019-08-08 15:19 EDT (History)
4 users (show)

See Also:


Attachments
Patch to move copy the workspace selection dialog into platform UI (24.07 KB, patch)
2010-03-09 10:15 EST, Francis Upton IV CLA
no flags Details | Diff
Some work - copying ConfigureWorkingsetAction (2.89 KB, patch)
2010-03-09 10:17 EST, Francis Upton IV CLA
no flags Details | Diff
Work in progress (36.78 KB, patch)
2010-04-25 06:25 EDT, Francis Upton IV CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kim Horne CLA 2006-09-19 14:13:45 EDT
I20060918-2000

If you set the Project Explorer to use working sets as top level elements the working set menu items (Select Working Set, Deselect Working Set, and Edit Working Set) become confusing.  Although the standard working set dialog does appear to work it's not at all clear that you're selecting a list of working sets to show.  The package explorer provides its own dialog for this purpose.  If you feel that we should be providing this dialog for all clients please open a new PR against platform-UI.
Comment 1 Michael D. Elder CLA 2006-09-19 15:20:01 EDT
What if the default dialog displayed a "Show as groups" checkbox?
Comment 2 Kim Horne CLA 2006-09-19 15:24:11 EDT
I dont think that would be applicable across the board - most viewers aren't configured to show working sets as elements.  That also doesn't address the additional menu items - deselect and edit are useless in this scenario.
Comment 3 Gunnar Wagenknecht CLA 2007-02-22 03:20:16 EST
I vote for using the dialog from the Package Explorer. The reason is that it also allows custom sorting of Working Sets which the default dialog doesn't provide.

There is also another usability issue. If you select Working Sets as top level elements the first time and no Working Sets are selected than ALL should be visible instead of an empty view. (3.3 M5) Maybe there should be a warning that no Working Sets are selected and the view will be empty.
Comment 4 Michael D. Elder CLA 2007-07-03 14:28:07 EDT
The behavior that Gunnar refers to has been fixed (showing all when no working sets are supported, but still exists for Window working sets - bug 190819)

Martin -- what are your thoughts on contributing the Working Set dialog to Platform?
Comment 5 Martin Aeschlimann CLA 2007-07-04 03:50:57 EDT
Yes, I think it would be good to have the working set selection dialog in platform as well. Platform already has a selection dialog, but it is more targeted to select a working set filter: You can choose 'Window working set' and 'No Working Set', which make no sense in our scenario.
So we could think of making the existing dialog more configurable.
Important is also that the selection dialog we need filters out some working sets (we only want Java and Resource working sets), and allows the artificial 'Other Projects' node.
So in the end it might be easier if you reimplement the dialog as there are so many special requirements here.
Comment 6 Mike Wilson CLA 2008-06-09 10:45:01 EDT
Not for 3.4.
Comment 7 Francis Upton IV CLA 2008-08-17 06:03:27 EDT
Kim, is this still an issue?  The package explorer and project explorer have identical dialogs for this now.  It does not seem confusing to me after the other working set bugs have been fixed (the omnibus working set bug 212389).
Comment 8 Kim Horne CLA 2008-08-20 12:35:44 EDT
(In reply to comment #7)
> Kim, is this still an issue?  The package explorer and project explorer have
> identical dialogs for this now.  It does not seem confusing to me after the
> other working set bugs have been fixed (the omnibus working set bug 212389).
> 

This is still an issue.  When the project explorer is in working set mode the menu items still appear.
Comment 9 Francis Upton IV CLA 2008-08-20 12:39:33 EDT
(In reply to comment #8)

> This is still an issue.  When the project explorer is in working set mode the
> menu items still appear.
> 
Yes, I agree that they appear, but I'm not confused by them.  I mean it seems that  you should be able to select or edit working sets even if the top level objects are working sets, so I don't understand the confusion.  Can you help me here?

Comment 10 Kim Horne CLA 2008-08-22 09:58:09 EDT
(In reply to comment #9)
> (In reply to comment #8)
> 
> > This is still an issue.  When the project explorer is in working set mode the
> > menu items still appear.
> > 
> Yes, I agree that they appear, but I'm not confused by them.  I mean it seems
> that  you should be able to select or edit working sets even if the top level
> objects are working sets, so I don't understand the confusion.  Can you help me
> here?
> 

They just dont make sense... if the top level elements are working sets, it's not a matter of selecting a working set, or even a composite working set, it's a matter of selecting N different working sets.  Similarly, "deselect" working set doesn't make any sense.  "Edit"  might, but even that's debatable.  This mode of work really does benefit from a specialized dialog such as the one JDT offers - "Configure Working Sets"
Comment 11 Francis Upton IV CLA 2008-08-22 10:09:40 EDT
I get it now, sorry for being so thick.  If it behaved as the Package Explorer
then it would make more sense.  I will make it so.
Comment 12 Dani Megert CLA 2009-01-30 02:17:38 EST
Francis, I agree with Kim and I don't see it as an enhancement.
Comment 13 Francis Upton IV CLA 2010-03-09 10:15:02 EST
Created attachment 161472 [details]
Patch to move copy the workspace selection dialog into platform UI

This is copied from the Java version and the Java-specific stuff was removed, but parameterized so that later JDT should be able to hook up to it.
Comment 14 Francis Upton IV CLA 2010-03-09 10:17:08 EST
Created attachment 161473 [details]
Some work - copying ConfigureWorkingsetAction

Does not compile, just saving it here.
Comment 15 Francis Upton IV CLA 2010-04-25 06:25:26 EDT
Created attachment 166015 [details]
Work in progress

Not ready yet, just saving it here.
Comment 16 Francis Upton IV CLA 2010-04-25 13:49:20 EDT
Close, but not going to make it this time around. Look for this in 3.7.
Comment 17 Eclipse Genie CLA 2018-09-15 00:31:16 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.
Comment 18 Lars Vogel CLA 2019-08-08 15:19:12 EDT
This bug has been marked with as "stalebug" for some time now. I mark this bug as "worksforme" to indicate that no work is planned here. 

If this bug is still relevant for the latest release, please reopen the bug and remote the "stalebug" whiteboard tag.