Attachment 153149 Details for Bug 296085 – Patch v01

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

[patch] Patch v01

Table_miss_1st_colmn.txt (text/plain), 1.29 KB, created by Praveen

on 2009-11-26 02:12:29 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Praveen

Created: 2009-11-26 02:12:29 EST

Size: 1.29 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.swt
>Index: Eclipse SWT/gtk/org/eclipse/swt/widgets/Table.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/Table.java,v
>retrieving revision 1.280
>diff -u -r1.280 Table.java
>--- Eclipse SWT/gtk/org/eclipse/swt/widgets/Table.java	18 Nov 2009 23:38:49 -0000	1.280
>+++ Eclipse SWT/gtk/org/eclipse/swt/widgets/Table.java	26 Nov 2009 07:06:29 -0000
>@@ -2046,13 +2046,19 @@
> 		}
> 		list = OS.g_list_next (list);
> 	}
>-	int /*long*/ columnHandle = 0;
>+	int /*long*/ columnHandle = OS.g_list_data (columns);
> 	if (fixVisible) {
>-		columnHandle = OS.g_list_data (columns);
> 		OS.gtk_tree_view_column_set_visible (columnHandle, true);
> 	}
> 	super.gtk_widget_size_request (widget, requisition);
>-	if (fixVisible) {
>+	/*
>+	* While the size is allocated for Table, there is a 
>+	* chance that the table column has the width set (and turned 
>+	* visible) in SetData listener code. In that case, the 
>+	* column's visibility should not set back to false.
>+	*/
>+	int columnWidth = OS.gtk_tree_view_column_get_fixed_width (columnHandle);
>+	if (fixVisible && columnWidth == 0) {
> 		OS.gtk_tree_view_column_set_visible (columnHandle, false);
> 	}
> 	if (columns != 0) OS.g_list_free (columns);

Actions: View | Diff

Attachments on bug 296085: 153149