Bug 520674 - [9][package explorer] decorate exported packages
Summary: [9][package explorer] decorate exported packages
Status: NEW
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.7   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: JDT-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-08-08 07:20 EDT by Stephan Herrmann CLA
Modified: 2018-11-19 18:53 EST (History)
4 users (show)

See Also:

Attachments
package with export decoration (600 bytes, image/png)
2018-01-16 16:53 EST, Stephan Herrmann CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Stephan Herrmann CLA 2017-08-08 07:20:34 EDT 
Wouldn't it be cool, if packages that are exported by their module be decorated in the package explorer (and perhaps elsewhere, too)?

Given that the module-info node in package explorer has no children (as IModuleDescription has no children), decorations seem a suitable way of displaying the export information.


In bug 519444 I have a draft icon for add-export, which could be standardized for everything relating to package exports (declared in module-info or via --add-exports).

Alternatively, we could use a "private" overlay to mark non-exported packages. Or, still, use a lock icon similar to Access rules nodes in Java Build Path.
Comment 1 Dani Megert CLA 2017-08-08 08:56:08 EDT 
Interesting idea. Something that needs a bit more thinking also in combination with OSGi package exports.

Suggest to look at this post Oxygen 1a.
Comment 2 Stephan Herrmann CLA 2017-08-10 07:36:45 EDT 
(In reply to Dani Megert from comment #1)
> Interesting idea. Something that needs a bit more thinking also in
> combination with OSGi package exports.
> 
> Suggest to look at this post Oxygen 1a.

Putting it on the radar for M4 :)
Comment 3 Stephan Herrmann CLA 2017-12-03 17:51:02 EST 
I'm running out of time for M4 - bulk move to M5
Comment 4 Stephan Herrmann CLA 2018-01-16 16:53:27 EST 
Created attachment 272292 [details]
package with export decoration

This is the icon that I was speaking of, nothing fancy, but a start for discussion :)
Comment 5 Stephan Herrmann CLA 2018-01-16 16:54:08 EST 
Not for M5, though ...