Bug 335096

Comment 1 Scott Kovatch 2011-02-02 15:04:40 EST

Subtracting out the checkbox column width in Table#getClientArea() fixes the problem, but it doesn't make sense to me why that should be the case. 

Adding Grant, since he did Table and Tree -- are checkboxes considered to be part of the 'client area' on Win32 and elsewhere?

The TableLayout code has a lot of platform-specific funkiness that may be part of the problem here, but I thought I'd ask before I check this in.

Comment 2 Grant Gayed 2011-02-04 12:12:52 EST

Yes, checkboxes are part of the client area on all platforms, snippet:

public static void main(String[] args) {
    final Display display = new Display();
    Shell shell = new Shell(display);
    shell.setLayout(new FillLayout());
    shell.setBounds(10,10,600,600);
    final Table table = new Table(shell, SWT.CHECK);
    new TableColumn(table, SWT.NONE).setWidth(100);
    new TableColumn(table, SWT.NONE).setWidth(100);
    new TableColumn(table, SWT.NONE).setWidth(100);
    table.setLinesVisible(true);
    for (int i = 0; i < 9; i++) {
        new TableItem(table, SWT.NONE).setText("item " + i);
    }
    table.addListener(SWT.Paint, new Listener() {
        public void handleEvent(Event event) {
            Rectangle rect = table.getClientArea();
            rect.width -= 1; rect.height -= 1;
            event.gc.setForeground(display.getSystemColor(SWT.COLOR_RED));
            event.gc.drawRectangle(rect);
        }
    });
    shell.open();
    while (!shell.isDisposed()) {
        if (!display.readAndDispatch()) display.sleep();
    }
    display.dispose();
}

Comment 3 Scott Kovatch 2011-02-04 12:39:44 EST

(In reply to comment #2)
> Yes, checkboxes are part of the client area on all platforms, snippet:

Thanks, Grant. In fact, this snippet demonstrates that it's not an SWT problem at all. Moving to UI.

Comment 4 Brian de Alwis 2011-03-29 10:55:11 EDT

Sorry for the delay in getting back to this.  I finally found a moment to look at fixing the JFace {Table,Tree}ColumnLayouts.  

We need information from SWT to determine the checkbox widths as the column widths on tables do not include the checkbox widths.

To see the problem, try adding the following listener to Grant's example.  It attempts to set the columns to a consistent width based on the available space.

	    table.addListener(SWT.Resize, new Listener() {
	        public void handleEvent(Event event) {
	        	int tableWidth = table.getClientArea().width;
	        	System.out.println("Table width: " + tableWidth);
	        	for(int i = 0; i < table.getColumnCount(); i++) {
	        		table.getColumn(i).setWidth(tableWidth / 3);
	        	}
	        }});

A table without SWT.CHECK never shows a horizontal scroll bar.
A table with SWT.CHECK always shows a horizontal scroll bar.

For the {Table,Tree}ColumnLayout to work properly for tables with checkboxes, we need to know the space taken by the checkboxes.  More accurately, we need to know the space that is actually available to the columns.  I suppose we could embed more platform-specific constants, but that's not very desirable.

Table and Tree on Cocoa do have a method getCheckColumnWidth(), but it is private.  I'm not sure about other platforms.  Would it be possible to expose these methods and make them API?

Comment 5 Eclipse Genie 2019-09-25 15:36:47 EDT

This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.