This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 319864 - Clean up dependency page of project descriptor
Summary: Clean up dependency page of project descriptor
Status: VERIFIED FIXED
Alias: None
Product: Tigerstripe (Archived)
Classification: Technology
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 enhancement (vote)
Target Milestone: 0.5M0   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-14 10:59 EDT by Navid Mehregani CLA
Modified: 2010-12-13 10:50 EST (History)
3 users (show)

See Also:


Attachments
Screenshot (55.95 KB, image/png)
2010-07-15 10:28 EDT, Navid Mehregani CLA
no flags Details
Patch_1 (88.22 KB, text/plain)
2010-12-10 07:48 EST, Valentin Yerastov CLA
no flags Details
icons (971 bytes, application/x-compressed-tar)
2010-12-10 07:49 EST, Valentin Yerastov CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Navid Mehregani CLA 2010-07-14 10:59:19 EDT
Build Identifier: 

Under dependency tab of a TS project, users can specify a list of referenced projects and a list of dependencies.  This is a bit confusing and misleading, since the referenced projects are also a dependency.  There is no logical reason for having two separate lists.

From what I understood from Richard is that this was done mostly due to historical reasons than anything else.  This defect is a general placeholder to clean up this page in the project descriptor.

My suggestion would be to remove the list of referenced projects and have a single list of dependencies where users can add jar, zip, or other TS projects.  The different type of dependencies can be distinguished with different icons.



Reproducible: Always
Comment 1 Eric Dillon CLA 2010-07-14 12:54:52 EDT
Also, as part of the clean up, the code related to the "Add standard built-in modules" should be removed.
This was put in place a few years back to address "built-in" modules which are now in place thru the "Installed Modules".
Comment 2 Navid Mehregani CLA 2010-07-15 10:28:51 EDT
Created attachment 174401 [details]
Screenshot
Comment 3 Navid Mehregani CLA 2010-11-22 09:53:04 EST
One thing to note is that exported TS modules only show up under dependencies list!
Comment 4 Valentin Yerastov CLA 2010-12-10 07:48:55 EST
Created attachment 184951 [details]
Patch_1
Comment 5 Valentin Yerastov CLA 2010-12-10 07:49:49 EST
Created attachment 184952 [details]
icons
Comment 6 Valentin Yerastov CLA 2010-12-10 07:51:36 EST
Done. If you have any problems with the icons I have attached a separate archive.
Comment 7 Yuri Strot CLA 2010-12-13 06:03:54 EST
Valentin's patch applied.
Comment 8 Navid Mehregani CLA 2010-12-13 10:50:30 EST
Looks great Valentin! Thanks!
I just changed the label from 'Referenced Tigerstripe Projects' to 'Dependencies', since the list can now hold dependencies of all types.