Bug 249065 - Preference to hide source bundles
Summary: Preference to hide source bundles
Status: RESOLVED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.5   Edit
Hardware: PC Windows Vista
: P3 normal (vote)
Target Milestone: 3.5 M7   Edit
Assignee: Chris Aniszczyk CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
: 264100 (view as bug list)
Depends on: 242168
Blocks:
  Show dependency tree
 
Reported: 2008-09-29 21:17 EDT by Jeff McAffer CLA
Modified: 2009-04-22 12:48 EDT (History)
4 users (show)

See Also:


Attachments
org.eclipse.pde.ui.patch (16.85 KB, patch)
2009-02-08 22:34 EST, Chris Aniszczyk CLA
no flags Details | Diff
org.eclipse.pde.ui.patch (20.55 KB, patch)
2009-02-09 11:39 EST, Chris Aniszczyk CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jeff McAffer CLA 2008-09-29 21:17:48 EDT
it would be great to have a preference in PDE to hide source bundles from all the various views.  It is not clear where people need/want to see them in the launch config or the plugins view or ...  An alternative would be to group them together under a different node in the view/dialog/list if possible.
Comment 1 Darin Wright CLA 2008-09-29 22:18:24 EDT
Yes, there is a general work item for this on the 3.5 plan. 
Comment 2 Curtis Windatt CLA 2008-09-30 09:20:05 EDT
Bug for suppressing them in the import dialog is bug 242168.
Comment 3 Curtis Windatt CLA 2009-01-06 16:41:44 EST
This will be looked at after most of the target platform rework is done, marking as M6.
Comment 4 Chris Aniszczyk CLA 2009-02-08 17:10:53 EST
*** Bug 264100 has been marked as a duplicate of this bug. ***
Comment 5 Chris Aniszczyk CLA 2009-02-08 22:34:28 EST
Created attachment 125088 [details]
org.eclipse.pde.ui.patch

A quick stab at adding the ability to hide source bundles. This enables it for the plug-ins view and PDE-related launch configurations.
Comment 6 Chris Aniszczyk CLA 2009-02-09 11:39:31 EST
Created attachment 125147 [details]
org.eclipse.pde.ui.patch

An updated patch.
Comment 7 Chris Aniszczyk CLA 2009-02-09 11:52:15 EST
done.

> 20090209

We added a new preference to show source bundles... it's off by default.

Source bundles are currently hidden in the plug-ins view and all PDE-based launch configurations.

There are still places we need to look at hiding source bundles, which I'll open another bug for.
Comment 8 Darin Wright CLA 2009-02-10 14:25:44 EST
Re-opening. I think the option is confusing in its current state. The current "Show source plug-ins" label/title doesn't let the user know where this will take effect. (Sure, we could add doc...)

I wonder if it would be better to have options in specific views for this. For example, in the new target editor, we want to show source bundles so users see what source will be part of their target (for debugging purposes). This will conflict with the default setting. We plan to have a toggle filter on the plug-ins (content) veiwer to show/hide source. In general, it might be simpler to have individual settings rather than a global preference (it also makes it easier to change on the fly rather than having to go back to the preference page to change a global setting).
Comment 9 Curtis Windatt CLA 2009-02-10 14:32:56 EST
(In reply to comment #8)
> In general, it might be simpler
> to have individual settings rather than a global preference (it also makes it
> easier to change on the fly rather than having to go back to the preference
> page to change a global setting).
> 

+1 for individual settings.  This would allow us to set proper defaults based on what we are doing.  For example, on the import dialog we would not show source by default, while in the target platform we would.  Right now, the default setting is to not show source bundles which just removes things randomly from the UI with no information on how to turn it back on (for example, in the launch config if you have 100 bundles, it says you have 200 of 200 plug-ins selected).

Comment 10 Chris Aniszczyk CLA 2009-02-10 14:45:23 EST
-1 for individual settings because 99.9% of the time people don't want to see source bundles. The only case I can envision is if people are looking to see what source locations are available.

We can discuss on the PDE call.
Comment 11 Darin Wright CLA 2009-02-10 14:54:43 EST
(In reply to comment #10)
> The only case I can envision is if people are looking to see
> what source locations are available.

I agree this is the most common case. To me the issues are that the preference in its current form does not have enough information to let the user know what checking/unchecking will do, and that this setting will conflict/not work on the target wizard (which feels inconsistent).

I agree we need ways to filter source bundles from the UI.

Comment 12 Chris Aniszczyk CLA 2009-02-10 15:56:08 EST
So we can improve the wording a bit.

On the PDE call, we discussed modifying IPluginModelBase to have an isSourceBundle or something to make filtering easier. I haven't decided yet if this is the best way to do it yet.
Comment 13 Jeff McAffer CLA 2009-02-21 22:06:19 EST
my 2c. In cases where the user "wants source" they probably don't want to see the source bundles in the list.  They just clutter things up.  For example, in the target platform people typically want the source bundles included if they are there but it is pretty uninteresting to see them in the list (other than to perhaps confirm that they are indeed selected).  In the launch or dependency scenarios source bundles are not relevant.

Summary: if there are no cases for wanting to see the source bundles, why have an option in the view/dialog to show the source bundles?  The global preference feels like a 'fail safe" option in case there is some scenario we are not seeing.  

In the target lists there might be an option to "include source bundles" or some such.
Comment 14 Chris Aniszczyk CLA 2009-03-10 18:46:15 EDT
we will polish this off in M7
Comment 15 Darin Wright CLA 2009-04-22 12:48:31 EDT
Marking fixed, as the preference is there and working. Will file new bugs as required.