| Summary: | [type hierarchy] view must honor package name abbreviation setting | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] JDT | Reporter: | Deepak Azad <deepakazad> | ||||||||
| Component: | UI | Assignee: | Markus Keller <markus.kell.r> | ||||||||
| Status: | VERIFIED FIXED | QA Contact: | |||||||||
| Severity: | normal | ||||||||||
| Priority: | P3 | CC: | daniel_megert, markus.kell.r, raksha.vasisht, rthakkar | ||||||||
| Version: | 3.7 | ||||||||||
| Target Milestone: | 3.7 M4 | ||||||||||
| Hardware: | All | ||||||||||
| OS: | All | ||||||||||
| Whiteboard: | |||||||||||
| Attachments: |
|
||||||||||
I fully agree, fixed in HierarchyLabelProvider. I've also fixed it in the quick hierarchy (in HierarchyInformationControl). The difference there is that we use the UI string to match against the filter pattern, so the behavior of the filter changes as well (which is good, IMO). Please speak up if you disagree. Thanks for the fix! (In reply to comment #1) > I've also fixed it in the quick hierarchy (in HierarchyInformationControl). The > difference there is that we use the UI string to match against the filter > pattern, so the behavior of the filter changes as well (which is good, IMO). +1. This means faster filtering. Markus can you please fix it in Ctrl+O Ctrl+O as well, so that the behavior is consistent everywhere. Created attachment 182641 [details]
screenshot after initial fix
Markus, the original problem I highlighted is fixed. But the setting is still not respected in 2 other places in the view (see screenshot)
Reopening... Created attachment 182644 [details]
More fixes
Fixes in Quick Outline, history dialogs, view descriptions, and type hierarchy members viewer.
Fixed in HEAD. Please add this item to the test plan, especially to test the filtering part. Verified for 3.7M4 with I20101206-1800. Verified in I20101206-1800. |
Created attachment 182620 [details] screenshot I use package name abbreviation to make it easier to parse information in Java views, and to reduce clutter. But Type Hierarchy view does not respect this setting... this is irritating (Call Hierarchy works correctly) Steps - Enable 'Show qualified type names' from the Type Hierarchy view menu - Enable package name abbreviation - Open type hierarchy on something