diff options
author | Anna Koskinen <Ansku@users.noreply.github.com> | 2021-08-27 14:01:08 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-08-27 14:01:08 +0300 |
commit | 4806947e508eee2006b2393a8cb6305eecd4e128 (patch) | |
tree | 80e5356a4bdeea4ac9c98271611177f09245401e | |
parent | c83f6bf042376dd5cedd9dc6af5f83683bd27e5f (diff) | |
download | vaadin-framework-4806947e508eee2006b2393a8cb6305eecd4e128.tar.gz vaadin-framework-4806947e508eee2006b2393a8cb6305eecd4e128.zip |
Applied save actions to compatibility Grid client-side classes. (#12389)8.14.0.beta1
9 files changed, 107 insertions, 96 deletions
diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/ColumnConfiguration.java b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/ColumnConfiguration.java index f9d117875a..6679cd2439 100644 --- a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/ColumnConfiguration.java +++ b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/ColumnConfiguration.java @@ -172,8 +172,8 @@ public interface ColumnConfiguration { throws IllegalArgumentException; /** - * Refreshes a range of columns in the current row containers in each Escalator - * section. + * Refreshes a range of columns in the current row containers in each + * Escalator section. * <p> * The data for the refreshed columns is queried from the current cell * renderer. diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/EscalatorUpdater.java b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/EscalatorUpdater.java index 893b1e1806..dd5249341d 100644 --- a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/EscalatorUpdater.java +++ b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/EscalatorUpdater.java @@ -26,9 +26,9 @@ import com.vaadin.v7.client.widgets.Escalator; * The updater is responsible for internally handling all remote communication, * should the displayed data need to be fetched remotely. * <p> - * This has a similar function to {@link com.vaadin.v7.client.widgets.Grid Grid's} - * {@link com.vaadin.v7.client.renderers.Renderer Renderers} - * , although they operate on different abstraction levels. + * This has a similar function to {@link com.vaadin.v7.client.widgets.Grid + * Grid's} {@link com.vaadin.v7.client.renderers.Renderer Renderers} , although + * they operate on different abstraction levels. * * @since 7.4 * @author Vaadin Ltd diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/events/SpacerVisibilityChangedEvent.java b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/events/SpacerVisibilityChangedEvent.java index 728fbd7eef..9323f2882e 100755 --- a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/events/SpacerVisibilityChangedEvent.java +++ b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/events/SpacerVisibilityChangedEvent.java @@ -42,10 +42,10 @@ public class SpacerVisibilityChangedEvent * Creates a spacer visibility changed event. * * @param rowIndex - * index of row to which the spacer belongs + * index of row to which the spacer belongs * @param visible - * {@code true} if the spacer element is shown, {@code false} if the - * spacer element is hidden + * {@code true} if the spacer element is shown, {@code false} if + * the spacer element is hidden */ public SpacerVisibilityChangedEvent(int rowIndex, boolean visible) { this.rowIndex = rowIndex; @@ -65,7 +65,7 @@ public class SpacerVisibilityChangedEvent * Gets whether the spacer element is displayed. * * @return {@code true} if the spacer element is shown, {@code false} if the - * spacer element is hidden + * spacer element is hidden */ public boolean isVisible() { return visible; diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/AutoScroller.java b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/AutoScroller.java index 73f6dd6716..370bd8b70a 100644 --- a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/AutoScroller.java +++ b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/AutoScroller.java @@ -355,8 +355,9 @@ public class AutoScroller { endBound = Math.max(finalEndBound, pageCordinate); } else { /* - * Subsequent runs make sure that the scroll area grows (but doesn't - * shrink) with the finger, but no further than the final bound. + * Subsequent runs make sure that the scroll area grows (but + * doesn't shrink) with the finger, but no further than the + * final bound. */ int oldTopBound = startBound; if (startBound < finalStartBound) { diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/DataAvailableEvent.java b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/DataAvailableEvent.java index 3e0b5c7572..f86d1639fb 100644 --- a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/DataAvailableEvent.java +++ b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/DataAvailableEvent.java @@ -34,7 +34,8 @@ public class DataAvailableEvent extends GwtEvent<DataAvailableHandler> { } /** - * Returns the range of available rows in {@link com.vaadin.client.data.DataSource DataSource} for this event. + * Returns the range of available rows in + * {@link com.vaadin.client.data.DataSource DataSource} for this event. * * @return range of available rows */ diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/selection/AbstractRowHandleSelectionModel.java b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/selection/AbstractRowHandleSelectionModel.java index 492ddbabc2..2038913f86 100644 --- a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/selection/AbstractRowHandleSelectionModel.java +++ b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/selection/AbstractRowHandleSelectionModel.java @@ -39,8 +39,7 @@ public abstract class AbstractRowHandleSelectionModel<T> * * @param handle * the handle to select by - * @return <code>true</code> if the selection state was changed by this - * call + * @return <code>true</code> if the selection state was changed by this call * @throws UnsupportedOperationException * if the selection model does not support either handles or * selection @@ -55,8 +54,7 @@ public abstract class AbstractRowHandleSelectionModel<T> * * @param handle * the handle to deselect by - * @return <code>true</code> if the selection state was changed by this - * call + * @return <code>true</code> if the selection state was changed by this call * @throws UnsupportedOperationException * if the selection model does not support either handles or * deselection diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/selection/MultiSelectionRenderer.java b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/selection/MultiSelectionRenderer.java index 33bae97e2c..e8b03011d2 100644 --- a/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/selection/MultiSelectionRenderer.java +++ b/compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/selection/MultiSelectionRenderer.java @@ -434,8 +434,9 @@ public class MultiSelectionRenderer<T> bottomBound = Math.max(finalBottomBound, pageY); } else { /* - * Subsequent runs make sure that the scroll area grows (but doesn't - * shrink) with the finger, but no further than the final bound. + * Subsequent runs make sure that the scroll area grows (but + * doesn't shrink) with the finger, but no further than the + * final bound. */ int oldTopBound = topBound; if (topBound < finalTopBound) { diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/widgets/Escalator.java b/compatibility-client/src/main/java/com/vaadin/v7/client/widgets/Escalator.java index a16c2be44d..a412bb9cec 100755 --- a/compatibility-client/src/main/java/com/vaadin/v7/client/widgets/Escalator.java +++ b/compatibility-client/src/main/java/com/vaadin/v7/client/widgets/Escalator.java @@ -392,8 +392,7 @@ public class Escalator extends Widget // Two movement objects for storing status and processing touches private Movement yMov, xMov; static final double MIN_VEL = 0.6, MAX_VEL = 4, F_VEL = 1500, - F_ACC = 0.7, - F_AXIS = 1; + F_ACC = 0.7, F_AXIS = 1; // The object to deal with one direction scrolling private class Movement { @@ -705,13 +704,13 @@ public class Escalator extends Widget /*-{ var vScroll = esc.@com.vaadin.v7.client.widgets.Escalator::verticalScrollbar; var vScrollElem = vScroll.@com.vaadin.v7.client.widget.escalator.ScrollbarBundle::getElement()(); - + var hScroll = esc.@com.vaadin.v7.client.widgets.Escalator::horizontalScrollbar; var hScrollElem = hScroll.@com.vaadin.v7.client.widget.escalator.ScrollbarBundle::getElement()(); - + return $entry(function(e) { var target = e.target; - + // in case the scroll event was native (i.e. scrollbars were dragged, or // the scrollTop/Left was manually modified), the bundles have old cache // values. We need to make sure that the caches are kept up to date. @@ -732,29 +731,29 @@ public class Escalator extends Widget return $entry(function(e) { var deltaX = e.deltaX ? e.deltaX : -0.5*e.wheelDeltaX; var deltaY = e.deltaY ? e.deltaY : -0.5*e.wheelDeltaY; - + // Delta mode 0 is in pixels; we don't need to do anything... - + // A delta mode of 1 means we're scrolling by lines instead of pixels // We need to scale the number of lines by the default line height if (e.deltaMode === 1) { var brc = esc.@com.vaadin.v7.client.widgets.Escalator::body; deltaY *= brc.@com.vaadin.v7.client.widgets.Escalator.AbstractRowContainer::getDefaultRowHeight()(); } - + // Other delta modes aren't supported if ((e.deltaMode !== undefined) && (e.deltaMode >= 2 || e.deltaMode < 0)) { var msg = "Unsupported wheel delta mode \"" + e.deltaMode + "\""; - + // Print warning message esc.@com.vaadin.v7.client.widgets.Escalator::logWarning(*)(msg); } - + // IE8 has only delta y if (isNaN(deltaY)) { deltaY = -0.5*e.wheelDelta; } - + @com.vaadin.v7.client.widgets.Escalator.JsniUtil::moveScrollFromEvent(*)(esc, deltaX, deltaY, e); }); }-*/; @@ -4772,15 +4771,13 @@ public class Escalator extends Widget public void show() { getRootElement().getStyle().clearDisplay(); getDecoElement().getStyle().clearDisplay(); - Escalator.this.fireEvent( - new SpacerVisibilityChangedEvent(getRow(), true)); + fireEvent(new SpacerVisibilityChangedEvent(getRow(), true)); } public void hide() { getRootElement().getStyle().setDisplay(Display.NONE); getDecoElement().getStyle().setDisplay(Display.NONE); - Escalator.this.fireEvent( - new SpacerVisibilityChangedEvent(getRow(), false)); + fireEvent(new SpacerVisibilityChangedEvent(getRow(), false)); } /** diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/widgets/Grid.java b/compatibility-client/src/main/java/com/vaadin/v7/client/widgets/Grid.java index 717acc1793..efd50e3d1e 100755 --- a/compatibility-client/src/main/java/com/vaadin/v7/client/widgets/Grid.java +++ b/compatibility-client/src/main/java/com/vaadin/v7/client/widgets/Grid.java @@ -186,7 +186,6 @@ import com.vaadin.v7.client.widget.grid.sort.SortOrder; import com.vaadin.v7.client.widgets.Escalator.AbstractRowContainer; import com.vaadin.v7.client.widgets.Escalator.SubPartArguments; import com.vaadin.v7.client.widgets.Grid.Editor.State; -import com.vaadin.v7.client.widgets.Grid.StaticSection.StaticCell; import com.vaadin.v7.client.widgets.Grid.StaticSection.StaticRow; import com.vaadin.v7.shared.ui.grid.ColumnResizeMode; import com.vaadin.v7.shared.ui.grid.GridConstants; @@ -272,8 +271,8 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, * a plain text caption */ public void setText(String text) { - this.content = text; - this.type = GridStaticCellType.TEXT; + content = text; + type = GridStaticCellType.TEXT; section.requestSectionRefresh(); } @@ -350,8 +349,8 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, * The html content of the cell */ public void setHtml(String html) { - this.content = html; - this.type = GridStaticCellType.HTML; + content = html; + type = GridStaticCellType.HTML; section.requestSectionRefresh(); } @@ -383,16 +382,16 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, * null). */ public void setWidget(Widget widget) { - if (this.content == widget) { + if (content == widget) { return; } - if (this.content instanceof Widget) { + if (content instanceof Widget) { // Old widget in the cell, detach it first - section.getGrid().detachWidget((Widget) this.content); + section.getGrid().detachWidget((Widget) content); } - this.content = widget; - this.type = GridStaticCellType.WIDGET; + content = widget; + type = GridStaticCellType.WIDGET; section.requestSectionRefresh(); } @@ -433,9 +432,9 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, * @since 7.6.3 */ void detach() { - if (this.content instanceof Widget) { + if (content instanceof Widget) { // Widget in the cell, detach it - section.getGrid().detachWidget((Widget) this.content); + section.getGrid().detachWidget((Widget) content); } } } @@ -1325,8 +1324,8 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, public void execute() { Element focusedElement = WidgetUtil.getFocusedElement(); if (focusedElement == grid.getElement() - || focusedElement == Document.get().getBody() - || count > 2) { + || focusedElement == Document.get().getBody() + || count > 2) { focusColumn(focusedColumnIndexDOM); } else { ++count; @@ -2147,12 +2146,12 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, // with the grid itself and the position needs to be compensated for try { TableRowElement rowElement = grid.getEscalator().getBody() - .getRowElement(grid.getEditor().getRow()); + .getRowElement(grid.getEditor().getRow()); int rowLeft = rowElement.getAbsoluteLeft(); int editorLeft = cellWrapper.getAbsoluteLeft(); if (editorLeft != rowLeft + frozenWidth) { - cellWrapper.getStyle().setLeft(newLeft + rowLeft - editorLeft, - Unit.PX); + cellWrapper.getStyle() + .setLeft(newLeft + rowLeft - editorLeft, Unit.PX); } } catch (IllegalStateException e) { // IllegalStateException may occur if user has scrolled Grid so @@ -3202,8 +3201,8 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, if (!columns.contains(column)) { throw new IllegalArgumentException( - "Given column is not a column in this grid. " - + column.toString()); + "Given column is not a column in this grid. " + + column.toString()); } if (!column.isSortable()) { @@ -3531,7 +3530,7 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, final double widthPerRatio; int leftOver = 0; if (BrowserInfo.get().isIE8() || BrowserInfo.get().isIE9() - || BrowserInfo.getBrowserString().contains("PhantomJS")) { + || BrowserInfo.getBrowserString().contains("PhantomJS")) { // These browsers report subpixels as integers. this usually // results into issues.. widthPerRatio = (int) (pixelsToDistribute / totalRatios); @@ -4390,8 +4389,8 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, // Check if the drop marker shouldn't be shown at all else if (dropMarkerLeft < frozenColumnsWidth - || dropMarkerLeft > Math.min(rightBoundaryForDrag, - escalator.getInnerWidth()) + || dropMarkerLeft > Math.min(rightBoundaryForDrag, + escalator.getInnerWidth()) || dropMarkerLeft < 0) { dropMarkerLeft = -10000000; } @@ -4540,7 +4539,8 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, if (focusedColumnIndex == draggedColumnIndex) { // move with the dragged column int adjustedDropIndex = latestColumnDropIndex > draggedColumnIndex - ? latestColumnDropIndex - 1 : latestColumnDropIndex; + ? latestColumnDropIndex - 1 + : latestColumnDropIndex; // remove hidden columns from indexing adjustedDropIndex = getVisibleColumns() .indexOf(getColumn(adjustedDropIndex)); @@ -4775,7 +4775,7 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, public void render(RendererCellReference cell, Object data) { if (!warned && !(data instanceof String)) { getLogger().warning(Column.this.toString() + ": " - + DEFAULT_RENDERER_WARNING); + + DEFAULT_RENDERER_WARNING); warned = true; } @@ -4796,7 +4796,8 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, private Grid<T> grid; /** - * Width of column in pixels as {@link #setWidth(double)} has been called. + * Width of column in pixels as {@link #setWidth(double)} has been + * called. */ protected double widthUser = GridConstants.DEFAULT_COLUMN_WIDTH_PX; @@ -4855,7 +4856,6 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, */ protected int expandRatio = GridConstants.DEFAULT_EXPAND_RATIO; - /** * Constructs a new column with a simple TextRenderer. */ @@ -5736,7 +5736,7 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, if (renderer instanceof WidgetRenderer) { try { Widget w = WidgetUtil.findWidget( - cell.getElement().getFirstChildElement()); + cell.getElement().getFirstChildElement()); if (w != null) { // Logical detach @@ -5958,13 +5958,14 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, // Need to wait for column width recalculation // scheduled by setWidth() before firing the event - Scheduler.get().scheduleDeferred( - new ScheduledCommand() { - @Override - public void execute() { - fireEvent(new ColumnResizeEvent<T>(col)); - } - }); + Scheduler.get() + .scheduleDeferred(new ScheduledCommand() { + @Override + public void execute() { + fireEvent(new ColumnResizeEvent<T>( + col)); + } + }); } }; @@ -6478,17 +6479,16 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, addColumnsSkipSelectionColumnCheck(Arrays.asList(columns), count); } - /** * Checks the given column is valid to add at the given index. */ private void checkColumnIsValidToAdd(Column<?, T> column, int index) { if (column == this.selectionColumn) { throw new IllegalArgumentException( - "The selection column may not be added manually"); + "The selection column may not be added manually"); } else if (this.selectionColumn != null && index == 0) { throw new IllegalStateException("A column cannot be inserted " - + "before the selection column"); + + "before the selection column"); } } @@ -6519,17 +6519,18 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, */ public <C extends Column<?, T>> C addColumn(C column, int index) { checkColumnIsValidToAdd(column, index); - addColumnsSkipSelectionColumnCheck(Collections.singleton(column), index); + addColumnsSkipSelectionColumnCheck(Collections.singleton(column), + index); return column; } - - private <C extends Column<?, T>> void addColumnsSkipSelectionColumnCheck(Collection<C> columnCollection, int index) { + private <C extends Column<?, T>> void addColumnsSkipSelectionColumnCheck( + Collection<C> columnCollection, int index) { int visibleNewColumns = 0; int currentIndex = index; - //prevent updates of hiding toggles. - //it will be updated finally all at once. + // prevent updates of hiding toggles. + // it will be updated finally all at once. this.columnHider.hidingColumn = true; for (final Column<?, T> column : columnCollection) { @@ -6546,7 +6547,8 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, } } if (visibleNewColumns > 0) { - final ColumnConfiguration columnConfiguration = this.escalator.getColumnConfiguration(); + final ColumnConfiguration columnConfiguration = this.escalator + .getColumnConfiguration(); columnConfiguration.insertColumns(index, visibleNewColumns); } @@ -6562,7 +6564,7 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, } sinkEvents(events); } - //now we do the update of the hiding toggles. + // now we do the update of the hiding toggles. this.columnHider.hidingColumn = false; this.columnHider.updateTogglesOrder(); refreshHeader(); @@ -7051,28 +7053,35 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, } this.dataSource = dataSource; - changeHandler = dataSource.addDataChangeHandler(new DataChangeHandler() { + changeHandler = dataSource + .addDataChangeHandler(new DataChangeHandler() { @Override public void dataUpdated(int firstIndex, int numberOfItems) { - escalator.getBody().refreshRows(firstIndex, numberOfItems); + escalator.getBody().refreshRows(firstIndex, + numberOfItems); } @Override public void dataRemoved(int firstIndex, int numberOfItems) { - escalator.getBody().removeRows(firstIndex, numberOfItems); - Range removed = Range.withLength(firstIndex, numberOfItems); + escalator.getBody().removeRows(firstIndex, + numberOfItems); + Range removed = Range.withLength(firstIndex, + numberOfItems); cellFocusHandler.rowsRemovedFromBody(removed); } @Override public void dataAdded(int firstIndex, int numberOfItems) { - escalator.getBody().insertRows(firstIndex, numberOfItems); - Range added = Range.withLength(firstIndex, numberOfItems); + escalator.getBody().insertRows(firstIndex, + numberOfItems); + Range added = Range.withLength(firstIndex, + numberOfItems); cellFocusHandler.rowsAddedToBody(added); } @Override - public void dataAvailable(int firstIndex, int numberOfItems) { + public void dataAvailable(int firstIndex, + int numberOfItems) { currentDataAvailable = Range.withLength(firstIndex, numberOfItems); fireEvent(new DataAvailableEvent(currentDataAvailable)); @@ -7084,28 +7093,31 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, int oldSize = body.getRowCount(); // Hide all details. - Set<Integer> oldDetails = new HashSet<Integer>(visibleDetails); + Set<Integer> oldDetails = new HashSet<Integer>( + visibleDetails); for (int i : oldDetails) { setDetailsVisible(i, false); } if (newSize > oldSize) { body.insertRows(oldSize, newSize - oldSize); - cellFocusHandler.rowsAddedToBody( - Range.withLength(oldSize, newSize - oldSize)); + cellFocusHandler.rowsAddedToBody(Range + .withLength(oldSize, newSize - oldSize)); } else if (newSize < oldSize) { body.removeRows(newSize, oldSize - newSize); - cellFocusHandler.rowsRemovedFromBody( - Range.withLength(newSize, oldSize - newSize)); + cellFocusHandler.rowsRemovedFromBody(Range + .withLength(newSize, oldSize - newSize)); } if (newSize > 0) { - Range visibleRowRange = escalator.getVisibleRowRange(); - dataSource.ensureAvailability(visibleRowRange.getStart(), + Range visibleRowRange = escalator + .getVisibleRowRange(); + dataSource.ensureAvailability( + visibleRowRange.getStart(), visibleRowRange.length()); } else { - // We won't expect any data more data updates, so just make - // the bookkeeping happy + // We won't expect any data more data updates, so + // just make the bookkeeping happy dataAvailable(0, 0); } @@ -7966,7 +7978,8 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, cellFocusHandler.offsetRangeBy(1); selectionColumn = new SelectionColumn(selectColumnRenderer); - addColumnsSkipSelectionColumnCheck(Collections.singleton(selectionColumn), 0); + addColumnsSkipSelectionColumnCheck( + Collections.singleton(selectionColumn), 0); selectionColumn.setEnabled(isEnabled()); selectionColumn.initDone(); @@ -8556,12 +8569,12 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>, * Adds a spacer visibility changed handler to the underlying escalator. * * @param handler - * the handler to be called when a spacer's visibility changes + * the handler to be called when a spacer's visibility changes * @return the registration object with which the handler can be removed * @since 7.7.13 */ public HandlerRegistration addSpacerVisibilityChangedHandler( - SpacerVisibilityChangedHandler handler) { + SpacerVisibilityChangedHandler handler) { return escalator.addHandler(handler, SpacerVisibilityChangedEvent.TYPE); } |