Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 316598 - [nebula grid] Making column header not visible causes first row to render incorrectly
Summary: [nebula grid] Making column header not visible causes first row to render inc...
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Nebula (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Chris Gross CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-11 08:45 EDT by Justin Dolezy CLA
Modified: 2021-07-05 11:39 EDT (History)
1 user (show)

See Also:

Attachments
Standard snippet modified to clearly demonstrate the problem (2.24 KB, application/octet-stream)
2010-06-11 08:47 EDT, Justin Dolezy CLA 		no flags Details
Patch that fixes the problem (1.16 KB, patch)
2010-06-11 09:23 EDT, Justin Dolezy CLA 		tom.schindl: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Justin Dolezy CLA 2010-06-11 08:45:49 EDT 
Build Identifier: I20100520-1744

If the first row is the same height as the column header, then when no column header is required the first row is not painted at all. If it is taller than the header then the portion where the column header would be is obscured.

Reproducible: Always

Steps to Reproduce:
1. Open GridSnippet1 (or the attached modified version)
2. Change setHeaderVisible() call to pass in false
3. Run and observe that not only is the header not visible, the first row isn't either
Comment 1 Justin Dolezy CLA 2010-06-11 08:47:45 EDT 
Created attachment 171716 [details]
Standard snippet modified to clearly demonstrate the problem
Comment 2 Justin Dolezy CLA 2010-06-11 09:23:50 EDT 
Created attachment 171719 [details]
Patch that fixes the problem

The code setting the clipping region does not take into account whether the column headers are visible or not.
Comment 3 Thomas Schindl CLA 2010-06-11 09:33:37 EDT 
prefect - released to head
Comment 4 Justin Dolezy CLA 2010-06-11 10:09:03 EDT 
Great! This will be in the Grid nightly build jar very soon then?
Comment 5 Thomas Schindl CLA 2010-06-11 10:11:16 EDT 
yes