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.

Bug 37360

Summary: table editor components should not have borders
Product: platform Reporter: madhav288 <madhav288>
Component: ExplorerAssignee: _ tboudreau <tboudreau>
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
table editor components should not have borders.
Please see the attachment.
Comment 1 madhav288 2003-11-20 08:05:59 UTC
Created attachment 12229 [details]
bug screenshot
Comment 2 madhav288 2003-11-20 08:11:41 UTC
Created attachment 12230 [details]
supposed behavior, without border for the text field.
Comment 3 _ tboudreau 2003-11-20 11:28:41 UTC
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

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.
Comment 4 _ tboudreau 2003-11-23 21:41:24 UTC
Hmm, after thinking about it, I've changed my mind.  Fixed on property
panel rewrite branch.
Comment 5 _ tboudreau 2003-12-09 09:36:54 UTC
Property panel rewrite branch merged.
Comment 6 Marian Mirilovic 2005-07-13 13:24:22 UTC