This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Summary: | Make color management UI elements reusable | ||
---|---|---|---|
Product: | editor | Reporter: | Theofanis Oikonomou <theofanis> |
Component: | Options | Assignee: | Theofanis Oikonomou <theofanis> |
Status: | RESOLVED WONTFIX | ||
Severity: | normal | CC: | apireviews, jglick, mkristofic |
Priority: | P3 | Keywords: | API_REVIEW_FAST |
Version: | 7.2 | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | ENHANCEMENT | Exception Reporter: | |
Attachments: |
options.editor PATCH
Old Terminal color management UI controls New Terminal color management UI controls |
Description
Theofanis Oikonomou
2012-01-23 16:18:53 UTC
Created attachment 115163 [details]
Old Terminal color management UI controls
Created attachment 115164 [details]
New Terminal color management UI controls
Y01 Terminal emulator does not have dependency on options.editor now. As terminal is editor independent, it does not make sense to make connection to editor APIs more tight. That would restrict re-use of the editor, while improving only options panel. Don't add dependency on more modules, use for example Y02 advice. Y02 The patch does not include the intended usage of the API, but if it is about the control to select right color, then: You can resule UI controls while having loosely coupled dependency on the other module via PropertyPanel and custom property editors: http://bits.netbeans.org/dev/javadoc/org-openide-explorer/org/openide/explorer/propertysheet/PropertyPanel.html http://bits.netbeans.org/dev/javadoc/org-openide-explorer/org/openide/explorer/doc-files/propertyViewCustomization.html (In reply to comment #3) > Y01 Terminal emulator does not have dependency on options.editor now. As > terminal is editor independent, it does not make sense to make connection to > editor APIs more tight. That would restrict re-use of the editor, while > improving only options panel. Don't add dependency on more modules, use for > example Y02 advice. > Thanks for the comment, point taken. closing for now. |