Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 327357

Summary: [Tree] Column hiding does not properly hide column data
Product: [RT] RAP Reporter: Rob A. <roba>
Component: RWTAssignee: Project Inbox <rap-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P2 CC: austin.riddle, tbuschto
Version: 1.4   
Target Milestone: 1.4 M3   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on:    
Bug Blocks: 320597    
Attachments:
Description Flags
Workbench Demo Patch Test Case
none
Screen shots showing display bug none

Description Rob A. CLA 2010-10-08 13:39:14 EDT 
Build Identifier: 20100917-0705

Hiding a column using a Tree widget by setting one or more columns to size 0 removes the column (including header), but data from all hidden column(s) are displayed and concatenated together in the last column of the table (the one without a header).   I tested using the Table widget and the same code works as expected.

I modified and attached the workbench demo application DemoTreeViewPart view to illustrate this behavior.  Replace the DemoTreeViewPart.java file with the attached version and launch the Workbench Demo Application.  Expand all child nodes for any of the tree views (View V probably the best) and enlarge the view so you can see all 4 columns displayed.  Then right click on any table cell and deselect the check Marked Column 3.  This results in column 3 being hidden and notice the data is still displayed.  Repeat steps and hid column 4 and you will see that column 4 is hidden but the last column in the tree contains data from column 3 & column 4.

Reproducible: Always

Steps to Reproduce:
1. Create a view container a TreeViewer containing 3 or more columns
2. Set the size of one or more columns to 0
3. Observe that the data is still displayed in last column
Comment 1 Rob A. CLA 2010-10-08 13:41:55 EDT 
Created attachment 180507 [details]
Workbench Demo Patch Test Case

Replace the DemoTreeViewPart.java file in the RAP Workbench Demo application with the attached file, then launch the Workbench Demo Application.  Expand all child
nodes for any of the tree views (View V probably the best) and enlarge the view
so you can see all 4 columns displayed.  Then right click on any table cell and
deselect the check Marked Column 3.  This results in column 3 being hidden and
notice the data is still displayed.  Repeat steps and hid column 4 and you will
see that column 4 is hidden but the last column in the tree contains data from
column 3 & column 4.
Comment 2 Rob A. CLA 2010-10-08 13:46:04 EDT 
Created attachment 180508 [details]
Screen shots showing display bug
Comment 3 Ivan Furnadjiev CLA 2010-10-11 03:08:51 EDT 
This issue can be reproduced with Controls Demo -> Tree Tab. If you resize the column to a very small width ( in the example above it is set to 0 to completely hide the column ), but the size that is less than the item left padding provoke the bug too.
Comment 4 Tim Buschtoens CLA 2010-10-11 12:02:42 EDT 
This is an issue of server-side layouting. A related bug can be seen with any image in a column except the first: It will not be cut-off if the column is to small to show it.
Comment 5 Tim Buschtoens CLA 2010-10-12 10:21:13 EDT 
Fixed in CVS HEAD in Tree.java. Also added and refactored a number of tests for Tree and TreeItem.
Comment 6 Austin Riddle CLA 2010-11-05 10:52:20 EDT 
*** Bug 328665 has been marked as a duplicate of this bug. ***