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: | Fold preview takes too much space | ||
---|---|---|---|
Product: | editor | Reporter: | Jiri Prox <jiriprox> |
Component: | Code folding | Assignee: | mslama <mslama> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | apireviews, jglick |
Priority: | P3 | Keywords: | API, API_REVIEW_FAST |
Version: | 6.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | |
Attachments: |
screen shot
Patch Patch with added constructor parameter |
Description
Jiri Prox
2009-12-08 03:41:38 UTC
Created attachment 92254 [details]
screen shot
Looks ugly. My guess is that the bottom part is the padding that we added some time ago to make editing at a position close to the end of a document slightly more convenient. It is regression because bottom padding was added to editor view. It should be disabled for tooltip view. Created attachment 92555 [details]
Patch
Marku, wouldn't it be possible to pass the boolean flag simply to the constructor of GapDocumentView? Having protected non-final field that's supposed to be modified by subclasses does not sound like a good API design to me. Created attachment 92601 [details]
Patch with added constructor parameter
jet-main #27febef72bc3 Field is made final to be able to set it just once in constructor. Integrated into 'main-golden', will be available in build *200912230201* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress) Changeset: http://hg.netbeans.org/main/rev/27febef72bc3 User: Marek Slama <mslama@netbeans.org> Log: #178284: Fold preview takes too much space. |