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

Bug 541720

Summary: [Gtk3] Regression: TableColumn.pack() clears header in 4.9
Product: [Eclipse Project] Platform Reporter: Patrick Tasse <patrick.tasse>
Component: SWTAssignee: Xi Yan <xixiyan>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: akurtakov, ericwill, loskutov
Version: 4.9Flags: akurtakov: review+
Target Milestone: 4.10 RC2   
Hardware: PC   
OS: Linux   
See Also: https://git.eclipse.org/r/133315
https://bugs.eclipse.org/bugs/show_bug.cgi?id=532941
https://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/?id=59b0f429a75003d153116d1b4753427591e69bad
Whiteboard:
Attachments:
Description Flags
Snippet
none
After packing one column
none
After packing all columns none

Description Patrick Tasse CLA 2018-11-29 17:05:38 EST
Created attachment 276769 [details]
Snippet

In 4.9, calling TableColumn.pack() clears the column header.

If all columns have their header cleared in this way, the header height is zeroed and the table header is no longer visible.

Packing the column from the OS (by double-clicking on the column header split) does not trigger the same problem.

The problem does not occur in 4.8.

Reproduced with gtk 3.10.8 and gtk 3.20.8.

Including a code snippet, double-clicking on a cell packs its column.
Comment 1 Patrick Tasse CLA 2018-11-29 17:06:42 EST
Created attachment 276770 [details]
After packing one column
Comment 2 Patrick Tasse CLA 2018-11-29 17:07:02 EST
Created attachment 276771 [details]
After packing all columns
Comment 3 Eclipse Genie CLA 2018-11-30 09:51:31 EST
New Gerrit change created: https://git.eclipse.org/r/133315
Comment 4 Andrey Loskutov CLA 2018-11-30 10:30:41 EST
Regression from bug 532941. I wonder if we can have a test which pack() the column twice and checks afterwards if it is still visible?
Comment 5 Andrey Loskutov CLA 2018-11-30 10:34:09 EST
Eric, can you please check the patch? I believe it should be OK for 4.10 RC2?
Comment 6 Patrick Tasse CLA 2018-11-30 10:58:59 EST
Thanks Xi, I tested your patch and it fixes the issue for me on gtk3.10.8.
Comment 8 Andrey Loskutov CLA 2018-12-02 04:59:44 EST
Alex, it is a bit too late, but this got merged after RC1. We need approval I guess.
Comment 9 Alexander Kurtakov CLA 2018-12-03 05:24:47 EST
Approved.
Please veirfy with latest I-build.
Comment 10 Andrey Loskutov CLA 2018-12-03 08:01:51 EST
Xi, Patrick, please verify with latest SDK build.
Comment 11 Patrick Tasse CLA 2018-12-03 10:44:35 EST
Tested in I20181203-0600 with gtk 3.10.8.
Comment 12 Xi Yan CLA 2018-12-03 12:19:58 EST
Verified in I20181203-0600 with GTK3.24.1.