Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 200832 - [context] Alt+click quick unfiltering don't show classpath containers when "Show 'Referenced Libraries' Node" is enabled in the Package Explorer
Summary: [context] Alt+click quick unfiltering don't show classpath containers when "S...
Status: CLOSED MOVED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 minor (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 201913 (view as bug list)
Depends on:
Blocks: 272089
  Show dependency tree
 
Reported: 2007-08-22 10:34 EDT by Eugene Kuleshov CLA
Modified: 2009-08-06 20:38 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Eugene Kuleshov CLA 2007-08-22 10:34:42 EDT
While in active task, quick unfiltering (Alt-click) on the project node in the Package Explorer view don't show classpath containers
Comment 1 Mik Kersten CLA 2007-08-24 00:36:39 EDT
Just investigated.  This is odd because the InterestFilter is not involved with determining whether that node should show, since all nodes given by the content provider should show when Alt+click is pressed.  So I think that the Java content provider is doing some tricks with determining when that node should be shown, since ClassPathContainer nodes are not a model element and may not be shown if no dependencies are to be shown.  Note that the Alt+click behavior works as expected for this case when using the Project Explorer.

Martin: can you say off-hand if there is a rule in the Package Explorer to do something along the lines of suppressing showing the Plug-in Dependencies node if none of it's children are going to be shown?
Comment 2 Eugene Kuleshov CLA 2007-08-24 00:42:13 EDT
I don't think it is a minor severity, because it doesn't allow to browse trough project dependencies. Also, I don't recall this in the old Eclipse, so it could be something introduced in one of the last 3.3 milestones.
Comment 3 Martin Aeschlimann CLA 2007-08-24 06:12:33 EDT
Yes, the Plug-in Container is only shown if there's at least one non-filtered child.
Comment 4 Eugene Kuleshov CLA 2007-08-31 13:07:40 EDT
Martin, would it be possible to replace this rule with a "visible" filter, that Mylyn can disable when needed?
Comment 5 Kim Sullivan CLA 2007-09-01 05:10:24 EDT
*** Bug 201913 has been marked as a duplicate of this bug. ***
Comment 6 Martin Aeschlimann CLA 2007-09-03 05:01:28 EDT
We added this to avoid that the container entries shows up alone when users filtered internal or/and external JARs.

What's your suggestion exactly, Eugene?
Comment 7 Eugene Kuleshov CLA 2007-09-03 11:11:01 EDT
(In reply to comment #6)
> We added this to avoid that the container entries shows up alone when users
> filtered internal or/and external JARs.
> 
> What's your suggestion exactly, Eugene?

I suggest to move that logic into a filter that can be disabled from view menu / Filters...  Then Mylyn can disable this filter automatically when focus on task context is on, and enable it back when focus is switched off.
Comment 8 Mik Kersten CLA 2007-09-11 01:08:09 EDT
(In reply to comment #7)
> I suggest to move that logic into a filter that can be disabled from view menu /
> Filters...  Then Mylyn can disable this filter automatically when focus on task
> context is on, and enable it back when focus is switched off.

-1.  I don't think that a filter that's an implementation detail (decision to hide a node when nothing is in it) should be exposed to the user in this way.

Martin: do you know off-hand what the difference is in the Package Explorer and Project Explorer's filtering/refresh policy with respect to filtering the Plug-in Dependencies container?  I believe that they both avoid showing the container if there are none, so I'm a bit confused about why the node is not showing.  In this scenario the Mylyn filter is always returning false for the select method so the expectation is that everything should be shown.  
Comment 9 Martin Aeschlimann CLA 2007-09-11 04:57:04 EDT
Only the package explorer tries to hide the referenced libraries and the classpath containers.
(This is hardcoded in:
org.eclipse.jdt.internal.ui.packageview.PackageExplorerPart.PackageExplorerProblemTreeViewer.isFiltered(Object, Object, ViewerFilter[]))

The project explorer always shows them. 

I'm not opposed to add a new filter 'Filter classpath containers'.

The 'Referenced Libraries' node is trickier as this depends on two filter settings and if there are class folders in the project.
Comment 10 Eugene Kuleshov CLA 2007-09-11 08:54:51 EDT
(In reply to comment #8)
> -1.  I don't think that a filter that's an implementation detail (decision to
> hide a node when nothing is in it) should be exposed to the user in this way.

Weird. I would say that filters are obvious candidates for hiding anything and they give much better visibility then hardcoded rules for hiding stuff
Comment 11 Eugene Kuleshov CLA 2008-02-15 12:00:10 EST
Martin, is there a chance that issue with "Referenced Libraries" and other empty classpath containers can be addressed in Eclipse 3.4? I.e. that rule converted to an explicitly visible filter?

Mik, I've also noticed that "Package Explorer" view also don't show non-empty classpath containers when project is Alt-Clicked, which is practically forces user to turn off Mylyn filtering on the entire view if they need to browse trough any jars in that container. That includes "Plug-in Dependencies" and "JRE System Library" classpath containers. Not sure why this is happening, but it might as well be that "Package Explorer" rules on hiding empty containers is getting confused by Mylyn's own filtering.
Comment 12 Martin Aeschlimann CLA 2008-02-18 06:30:55 EST
I would need a bug in JDT/UI describing exactly what you expect us to add and how it should work.
Comment 13 Eugene Kuleshov CLA 2008-02-18 13:16:39 EST
 (In reply to comment #12)
> I would need a bug in JDT/UI describing exactly what you expect us to add and
> how it should work.

I created bug 219335 for that. Hope that helps.
Comment 14 Mik Kersten CLA 2008-02-18 23:44:44 EST
Eugene: I assume that you only see the problem you state in comment#11 in the Package Explorer, and that it works as expected in the Project Explorer?  I took a quick look and am wondering if this is just another instance of bug 219335 that you created.
Comment 15 Eugene Kuleshov CLA 2008-02-19 00:31:44 EST
Mik, looking at the FAQ entry you've just created I now wonder if Mylyn could disable "Show Referenced Libraries Node" when "focus on active task" in the Package Explorer is on? Though I still 

(In reply to comment #14)
> Eugene: I assume that you only see the problem you state in comment#11 in the
> Package Explorer, and that it works as expected in the Project Explorer?  I took
> a quick look and am wondering if this is just another instance of bug 219335
> that you created.

Not sure what you mean by another instance and bug 219335 been created only to track things at JDT side as per Martin request.
I am not using Project Explorer, so can't say if it has the same issue or not.
Comment 16 Mik Kersten CLA 2008-06-12 18:17:22 EDT
Need to defer: http://wiki.eclipse.org/index.php/Mylyn/3.0_Plan#Deferred_Items
Comment 17 Eclipse Webmaster CLA 2022-11-15 11:45:08 EST
Mylyn has been restructured, and our issue tracking has moved to GitHub [1].

We are closing ~14K Bugzilla issues to give the new team a fresh start. If you feel that this issue is still relevant, please create a new one on GitHub.

[1] https://github.com/orgs/eclipse-mylyn