Bug 245258 - [WorkingSets] WorkingSetConfigurationBlock should support mode for selecting working sets as scope
Summary: [WorkingSets] WorkingSetConfigurationBlock should support mode for selecting ...
Status: CLOSED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.5   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform UI Triaged CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords:
Depends on:
Blocks: 39364 291568
  Show dependency tree
 
Reported: 2008-08-26 10:53 EDT by Markus Keller CLA
Modified: 2019-11-08 04:37 EST (History)
3 users (show)

See Also:


Attachments
work in progress (33.98 KB, patch)
2008-11-27 04:54 EST, Markus Keller CLA
no flags Details | Diff
Something like this? (54.80 KB, patch)
2008-12-03 07:09 EST, Hitesh CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Keller CLA 2008-08-26 10:53:24 EDT
I20080821-1005

To solve bug 39364, WorkingSetConfigurationBlock should support a mode for selecting working sets as scope of an operation. It should allow to choose the window working set and it should allow to modify existing and add new working sets.

The required functionality has been listed in bug 39364 comment 8:

> Hm, it's not that easy. The WorkingSetConfigurationBlock API ...
> 1. does not offer a way to present all available working set types

=> bug 245106

> 2. always adds a check box (e.g. for "Add project to working sets"), which
> must not show up in the search dialog
> 3. does not list the workbench window working set and does not allow the user
> to edit working sets

=> This could be implemented with a new constructor, e.g.
    WorkingSetConfigurationBlock(String[] workingSetIds,
            IDialogSettings settings, String comboLabel, String selectLabel),
which:
- suppresses the "Add project to working sets" checkbox
- creates a WorkingSetSelectionDialog instead of a SimpleWorkingSetSelectionDialog
Comment 1 Markus Keller CLA 2008-09-03 10:22:16 EDT
Hmm, I realized that we need a bit more than this to resolve bug 39364:

In the search dialog, we should not have label in front of the combo (the scope selection radio serves as label there).

We also need some notification when the working set has been changed (either via combo or via the button), because we have to select the "Working Set:" radio then. Maybe let WorkingSetConfigurationBlock implement ISelectionProvider.

I'll look into this.
Comment 2 Kim Horne CLA 2008-11-26 12:52:48 EST
Hitesh, could you please assist Markus with this?
Comment 3 Markus Keller CLA 2008-11-27 04:54:23 EST
Created attachment 118877 [details]
work in progress

I've already started with this, but I had to stop because the fix became bigger than I expected. If you want, you can use my work in progress. I won't have time to do this for M4.
Comment 4 Hitesh CLA 2008-12-03 07:09:10 EST
Created attachment 119367 [details]
Something like this? 

We could separate out the combo box and the button into another block and use it the configuration block and the search dialog. Something like this? 
Your thoughts ?
Comment 5 Boris Bokowski CLA 2009-05-06 16:50:31 EDT
Removing 3.5 target milestone. We are in the end-game now. Please have a look and decide if this should be targeted at 3.6.
Comment 6 Dani Megert CLA 2009-05-07 02:45:43 EDT
Given we already have some patches here, I suggest to target this for 3.6.
Comment 7 Hitesh CLA 2009-05-07 05:44:19 EDT
(In reply to comment #6)
> Given we already have some patches here, I suggest to target this for 3.6.
> 
Dani, this will need a closer look.

On reconsideration, I am not sure if either of the patches do anything significantly meaningful. 

Patch 1 :  The 'scope mode', unfortunately, hardly belongs to the context of  workingset selection .Try to think of the WorkingSetConfigurationBlock in general, not just from a search dialogs perspective.

Patch 2 : Tries to address the need by extracting a subset UI. Seems OK initially but , but the WorkingSetComboBlock widget is not very meaningful, although very useful.

Any thoughts?
 
Comment 8 Dani Megert CLA 2009-05-07 05:47:06 EDT
>On reconsideration, I am not sure if either of the patches do anything
>significantly meaningful. 
I didn't look at the patches. I only wanted to make sure the started work is not lost that's why I suggested to target it to 3.6. Definitely not something we should spend any cycles at this point.
Comment 9 Eric Moffatt CLA 2009-05-07 10:19:29 EDT
Yep, 3.6 it is...
Comment 10 Paul Webster CLA 2010-06-09 08:17:51 EDT
Removed from 3.6.  Owners can re-assess.

PW
Comment 11 Lars Vogel CLA 2019-11-08 04:37:39 EST
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.

If the bug is still relevant please remove the stalebug whiteboard tag.