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: | API for sticky windows | ||
|---|---|---|---|
| Product: | editor | Reporter: | Ralph Ruijs <ralphbenjamin> |
| Component: | Painting & Printing | Assignee: | Ralph Ruijs <ralphbenjamin> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | mentlicher, mkristofic, mmetelka, ralphbenjamin, sdedic |
| Priority: | P1 | Keywords: | API, API_REVIEW_FAST |
| Version: | 8.2 | ||
| Hardware: | All | ||
| OS: | All | ||
| Issue Type: | ENHANCEMENT | Exception Reporter: | |
| Bug Depends on: | 267807 | ||
| Bug Blocks: | 215073 | ||
| Attachments: | Proposed patch | ||
|
Description
Ralph Ruijs
2016-04-11 06:42:33 UTC
Created attachment 159197 [details]
Proposed patch
Please review the proposed api change. The code is on debugger_pin_watch branch in jet-main repository. Thanks! Ralph, apologies for delay. MM01) Since the editor.lib should be abandoned in the future I would appreciate if all the new editor apis could be placed in editor.lib2. It may be a bit tricky in this case but it should be doable since there's an implementation dependency of editor.lib on editor.lib2 so some necessary "bridges" could be added. MM02) If there's no reason to extend the StickyWindowSupport class let's make it final. The EditorUI.getStickyWindowSupport() could IMHO become StickyWindowSupport.get(JTextComponent). MM03) Although it's a mostly visual feature maybe some basic test could be added for the added methods? I did not read the debugger part of the change yet. Thanks. Mila jet-main#9b0af5705974 MM01: moved to editor.lib2 MM02: thanks for the catch, updated MM03: the methods forward everything to swing, even the convertPoint method. If there are no further objections, I'm going to merge the branch into trunk this evening CEST. The debugger_pin_watch branch is merged into trunk: http://hg.netbeans.org/jet-main/rev/407d58493c75 *** Bug 228015 has been marked as a duplicate of this bug. *** |