This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 274550 - [explorer] Clean up and streamline explorer UI
Summary: [explorer] Clean up and streamline explorer UI
Status: CLOSED FIXED
Alias: None
Product: JSDT
Classification: WebTools
Component: General (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows Vista
: P3 major (vote)
Target Milestone: 3.2 M7   Edit
Assignee: Nitin Dahyabhai CLA
QA Contact: Nitin Dahyabhai CLA
URL:
Whiteboard: PMC_approved
Keywords: plan
: 248779 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-04-30 12:54 EDT by Kaloyan Raev CLA
Modified: 2010-05-24 16:01 EDT (History)
3 users (show)

See Also:
david_williams: pmc_approved+
thatnitind: pmc_approved? (raghunathan.srinivasan)
thatnitind: pmc_approved? (naci.dai)
thatnitind: pmc_approved? (deboer)
thatnitind: pmc_approved? (neil.hauge)
thatnitind: pmc_approved? (kaloyan)


Attachments
screenshot (17.27 KB, image/png)
2009-04-30 12:54 EDT, Kaloyan Raev CLA
no flags Details
Not a complete solution, but a good start. (6.13 KB, patch)
2010-03-09 17:03 EST, Chris Jaun CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kaloyan Raev CLA 2009-04-30 12:54:19 EDT
Created attachment 133966 [details]
screenshot

See the attached screenshot. 

The WebContent folder of the Dynamic Web Project is decorated with the JavaScript decorator. 

The project structure has also the "JavaScript Resources" library node. 

The projects on the screenshot do not have the JavaScript Toolkit facet installed.
Comment 1 Nitin Dahyabhai CLA 2009-04-30 15:41:57 EDT
Are you sure the facet or nature is not installed?  Also, the WebContent folder is now a JavaScript source folder by default so that its contents are properly indexed.  We're still looking at the common navigator contributions with an eye on how to properly represent all of this to the user.
Comment 2 Chris Jaun CLA 2009-09-15 13:09:16 EDT
Categorizing JSDT bugzillas for planning purposes.
Comment 3 Nitin Dahyabhai CLA 2010-02-24 00:50:24 EST
Renaming the bug to better reflect plan:
1) Deprecate/hide Script Explorer in favor of the Project Explorer by default.
2) Stop interfering with the display of non-JS files.  Never interfere with the display of folders.
3) Make JS files on our include path optionally expandable to show their structure
4) Remove the flat/hierarchical layout offering from JSDT.
5) Reorganize the contents of the "JavaScript Resources" node in a similar manner to what's been done for "Java Resources".
6) Correct status line labels when a JS element is selected, e.g. a verbose Library "description" should not precede the element's identifier.
Comment 4 Chris Jaun CLA 2010-03-09 17:03:21 EST
Created attachment 161534 [details]
Not a complete solution, but a good start.

This patch does the following:

1. Sets WebContent as a source by default
2. Sets the default view in JavaScript Perspective to be the Project Explorer.
3. Removes the UI associated with source folders and packages. The JavaScript Resources node remains.
Comment 5 Chris Jaun CLA 2010-03-12 13:53:57 EST
*** Bug 248779 has been marked as a duplicate of this bug. ***
Comment 6 David Williams CLA 2010-04-07 00:11:17 EDT
Sounds like more than "clean up" ... all important. 
Thanks.
Comment 7 Nitin Dahyabhai CLA 2010-04-28 23:47:15 EDT
Fixed, except item 4 was made to work, and for item 6, which is still a problem and typified by bug 285758 and deferred to 3.3.
Comment 8 Nitin Dahyabhai CLA 2010-05-24 16:01:49 EDT
Closing.

1) Deprecate/hide Script Explorer in favor of the Project Explorer by default.
 Done.
2) Stop interfering with the display of non-JS files.  Never interfere with the display of folders.
 Done.
3) Make JS files on our include path optionally expandable to show their structure
 Done.
4) Remove the flat/hierarchical layout offering from JSDT.
 Reimplemented "hierarchical" layout as "Groups".
5) Reorganize the contents of the "JavaScript Resources" node in a similar manner to what's been done for "Java Resources".
 Source entries from the project Include Path now visible.
6) Correct status line labels when a JS element is selected, e.g. a verbose Library "description" should not precede the element's identifier.
 Deferred to bug 285758.