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

Bug 384124

Summary: Cell editors overlap Table header when scrolling (mouse wheel)
Product: [RT] RAP Reporter: r k <rkunitzky>
Component: RWTAssignee: Project Inbox <rap-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P1 CC: ivan, sgouta2, tbuschto
Version: unspecified   
Target Milestone: 2.0 M1   
Hardware: All   
OS: All   
Whiteboard: sr152
Attachments:
Description Flags
Image overlaps column header none

Description r k CLA 2012-07-03 06:00:05 EDT 
Build Identifier: Version: Indigo Service Release 1 Build id: 20110916-0149

As described in: http://www.eclipse.org/forums/index.php/t/366732/ 

I used the concept of TableViewer together with TabelEditors to place controls inside a tableviewerCell. It actually works perfect(I worked according to the eclipse example), but whenever the table is scrolled, the cells with controls (TableEditor) overlap the header (Columnname) of the table.

Reproducible: Always
Comment 1 Ivan Furnadjiev CLA 2012-07-03 06:16:01 EDT 
Reproducible with Controls Demo - TableViewer tab:
1. Add 100 items
2. Enable cell editors
3. Activate a cell editor and use mouse wheel to scroll down - cell editor is visible (or partly) on the Table header
Set the priority to P1 as this is a regression.
Comment 2 r k CLA 2012-07-03 11:32:15 EDT 
I have developed a workaround. I use a scrollbarlistener and the table topIndex together with a list of controls created during the table generation. 

Whenever the table is scrolled, it detects the topIndex and the lastTopIndex and calculates the amount of objects to hide or to show. 
The only input needed is the amount of controls p. row. 

It works fine for me.
Comment 3 Tim Buschtoens CLA 2012-07-04 04:36:49 EDT 
Fixed with commit c605c79f34312b0471e7e2cf881fa478865ccc66 by setting ZIndex of GridHeader.
Comment 4 Seif Gouta CLA 2012-10-23 12:40:59 EDT 
Created attachment 222682 [details]
Image overlaps column header
Comment 5 Seif Gouta CLA 2012-10-23 12:47:12 EDT 
Seems like we are getting the same problem again with RAP 1.5.1. Can we reopen this issue in that case? Will you manage to look at it without code snippet?
Comment 6 Seif Gouta CLA 2012-11-13 10:15:50 EST 
(In reply to comment #3)
> Fixed with commit c605c79f34312b0471e7e2cf881fa478865ccc66 by setting ZIndex
> of GridHeader.

Hello
I recreated this defect with RAP 1.5.1 but noticed that it is fixed in version 2.0.0.
We are currently using version 1.5.1, can I refer to this defect or should I create a new defect?
Regards,
Seif
Comment 7 Ivan Furnadjiev CLA 2012-11-15 08:45:52 EST 
(In reply to comment #6)
> (In reply to comment #3)
> > Fixed with commit c605c79f34312b0471e7e2cf881fa478865ccc66 by setting ZIndex
> > of GridHeader.
> 
> Hello
> I recreated this defect with RAP 1.5.1 but noticed that it is fixed in
> version 2.0.0.
> We are currently using version 1.5.1, can I refer to this defect or should I
> create a new defect?
> Regards,
> Seif

I will reopen this bug and we will consider to fix it for 1.5.2.
Comment 8 Ivan Furnadjiev CLA 2012-11-15 08:48:17 EST 
Keep status fixed, but mark it for consideration for 1.5.2
Comment 9 Ralf Sternberg CLA 2013-01-14 10:40:38 EST 
I'm fine with backporting the patch to 1.5.2, but for the master branch, I'd prefer to have a pointer to the bug, preferably in a comment with the test. Otherwise it won't be clear why the z-index is set to 2000.
Comment 10 Ivan Furnadjiev CLA 2013-01-22 03:25:33 EST 
Backported to 1.5-maintenance branch with commit 12e785443228094bc26748cc117d35ef2ba82c7b.