ColumnFormat now computes size correctly on mac.

Author: nedtwigg

durian-swt/src/main/java/com/diffplug/common/swt/ColumnFormat.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2020 DiffPlug
+ * Copyright (C) 2020-2021 DiffPlug
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -16,6 +16,7 @@
 package com.diffplug.common.swt;
 
 
+import com.diffplug.common.swt.os.OS;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
@@ -109,12 +110,43 @@ protected static Table buildTable(Composite parent, int style, boolean linesVisi
 		// create the columns and layout
 		Function<ColumnBuilder, TableColumn> buildFunc = builder -> builder.build(control);
 		List<TableColumn> columns = columnBuilders.stream().map(buildFunc).collect(Collectors.toList());
-		buildLayout(control, new TableColumnLayout(), columns, columnBuilders);
+
+		boolean needsSpecialLayout = SwtMisc.flagIsSet(SWT.CHECK, style) && OS.getNative().isMac();
+		TableColumnLayout layout = needsSpecialLayout ? new MacCheckTableColumnLayout() : new TableColumnLayout();
+		buildLayout(control, layout, columns, columnBuilders);
 
 		// return the control
 		return control;
 	}
 
+	/** Adds a phantom column for the checkboxes so that the rest of the space is calculated correctly. */
+	private static class MacCheckTableColumnLayout extends TableColumnLayout {
+		private static final int CHECK_COLUMN_WIDTH = 15;
+
+		@Override
+		protected int getColumnCount(Scrollable tableTree) {
+			return super.getColumnCount(tableTree) + 1;
+		}
+
+		@Override
+		protected void setColumnWidths(Scrollable tableTree, int[] widths) {
+			TableColumn[] columns = ((Table) tableTree).getColumns();
+			for (int i = 0; i < columns.length; i++) {
+				columns[i].setWidth(widths[i]);
+			}
+		}
+
+		@Override
+		protected ColumnLayoutData getLayoutData(Scrollable tableTree, int columnIndex) {
+			Table table = (Table) tableTree;
+			if (columnIndex < table.getColumnCount()) {
+				return (ColumnLayoutData) table.getColumn(columnIndex).getData(LAYOUT_DATA);
+			} else {
+				return new ColumnPixelData(CHECK_COLUMN_WIDTH);
+			}
+		}
+	}
+
 	/** Builds a table with the given columns. */
 	protected static Tree buildTree(Composite parent, int style, boolean linesVisible, boolean headerVisible, List<? extends ColumnBuilder> columnBuilders) {
 		SwtMisc.assertClean(parent);

durian-swt/src/test/java/com/diffplug/common/swt/jface/ColumnViewerFormatTest.java

@@ -0,0 +1,48 @@
+/*
+ * Copyright (C) 2020-2021 DiffPlug
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package com.diffplug.common.swt.jface;
+
+
+import com.diffplug.common.swt.InteractiveTest;
+import com.diffplug.common.swt.Layouts;
+import java.util.Arrays;
+import org.eclipse.jface.viewers.ArrayContentProvider;
+import org.eclipse.jface.viewers.TableViewer;
+import org.eclipse.swt.SWT;
+import org.eclipse.swt.widgets.Composite;
+import org.junit.Test;
+import org.junit.experimental.categories.Category;
+
+@Category(InteractiveTest.class)
+public class ColumnViewerFormatTest {
+	@Test
+	public void testCopy() {
+		InteractiveTest.testCoat("Column viewer", 10, 10, cmp -> {
+			Layouts.setFill(cmp);
+			buildTableWithCheck(new Composite(cmp, SWT.BORDER), true);
+			buildTableWithCheck(new Composite(cmp, SWT.BORDER), false);
+		});
+	}
+
+	private void buildTableWithCheck(Composite cmp, boolean hasCheck) {
+		ColumnViewerFormat<String> format = ColumnViewerFormat.builder();
+		format.setStyle(SWT.FULL_SELECTION | (hasCheck ? SWT.CHECK : SWT.NONE));
+		format.addColumn().setLabelProviderText(s -> s);
+		TableViewer viewer = format.buildTable(cmp);
+		viewer.setContentProvider(ArrayContentProvider.getInstance());
+		viewer.setInput(Arrays.asList("Onesy twosy threesy foursy", "red orange yellow green blue purple"));
+	}
+}