]> source.dussan.org Git - vaadin-framework.git/commit
Fixes a Grid issue that breaks sorting. (#10999)
authorduffrohde <rohde@duff.dk>
Tue, 2 Apr 2019 06:16:27 +0000 (08:16 +0200)
committerOlli Tietäväinen <ollit@vaadin.com>
Tue, 2 Apr 2019 06:16:27 +0000 (09:16 +0300)
commitdf19d2a4a5e5e44b0231fd4bc1119ad11f1c71ac
treed4be3ec448b07b35b30460fe3cb7a88e44f761fa
parenta0de6289f8b6e4ef0d95658f7b6d758174025f36
Fixes a Grid issue that breaks sorting. (#10999)

* Fixes a Grid issue that breaks sorting.

When multi-select has been enabled and disabled a number of times, sorting on the first column in a Grid is broken.
Unregistering a click handler on the header solves the problem.

* Added a test checking that sorting of a grid column still works after having enabled and then disabled multi select mode.
The grid is configured with an extra header row containing a text box since it triggers an exception if the header click handler for the multi select column is not removed.

* Remove an unused import that was added by mistake
client/src/main/java/com/vaadin/client/widgets/Grid.java
uitest/src/main/java/com/vaadin/tests/widgetset/client/grid/GridBasicClientFeaturesWidget.java
uitest/src/test/java/com/vaadin/tests/components/grid/basicfeatures/client/GridHeaderTest.java