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: | table editor components should not have borders | ||
---|---|---|---|
Product: | platform | Reporter: | madhav288 <madhav288> |
Component: | Explorer | Assignee: | _ tboudreau <tboudreau> |
Status: | VERIFIED FIXED | ||
Severity: | blocker | CC: | dpavlica |
Priority: | P3 | ||
Version: | 3.x | ||
Hardware: | PC | ||
OS: | Windows ME/2000 | ||
Issue Type: | DEFECT | Exception Reporter: | |
Bug Depends on: | 31896, 36719 | ||
Bug Blocks: | |||
Attachments: |
bug screenshot
supposed behavior, without border for the text field. |
Description
madhav288
2003-11-20 08:03:58 UTC
Created attachment 12229 [details]
bug screenshot
Created attachment 12230 [details]
supposed behavior, without border for the text field.
The real problem here is that the height of the field is too short. This has already been fixed on the property panel rewrite branch, which will be merged soon. In the current trunk, there are two problems that create this situation: - The height of columns is based on the max of the height of the expansion icon and the height of the font - it doesn't take into account borders. - Windows classic look and feel uses a two pixel wide border for text components. In fact, on metal or XP look and feels, it works fine in the trunk because the text field border is a single line border The point is that the text field border should be whatever the look and feel decides is the correct type of border - otherwise you can end up in situations where it is impossible to tell if an editor is present or not. So it's much safer to use the border that the look and feel supplies for whatever type of component you're using. It serves as an indication that the editor is indeed in edit mode. Once the property panel rewrite branch is merged, the height of the fields will take into account the border height, and this will be a non-problem even with the border, so I'm closing this issue as WONTFIX. See issue 36719 re the height of rows in the table. Adding Dusan from HIE to cc in case he has an opinion on this. Hmm, after thinking about it, I've changed my mind. Fixed on property panel rewrite branch. Property panel rewrite branch merged. closed |