diff options
author | Teemu Suo-Anttila <teemusa@vaadin.com> | 2015-12-17 15:56:43 +0200 |
---|---|---|
committer | Vaadin Code Review <review@vaadin.com> | 2015-12-18 10:45:56 +0000 |
commit | 6d6cd0df3c279f1c24ac370488a5b6993c8424d0 (patch) | |
tree | 97cdaffa2eebee2fd6a1140584a144f8888665db /client | |
parent | 7146d04d0727b90ad49e83d075fd277d46317e69 (diff) | |
download | vaadin-framework-6d6cd0df3c279f1c24ac370488a5b6993c8424d0.tar.gz vaadin-framework-6d6cd0df3c279f1c24ac370488a5b6993c8424d0.zip |
Verify Column width after all sort related cell changes (#16597)
This fixes an IE8 issue where multisorting is not shown correctly.
Change-Id: Id68103f06baf9e3ae20cb2714407b467829ef1a0
Diffstat (limited to 'client')
-rw-r--r-- | client/src/com/vaadin/client/widgets/Grid.java | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/client/src/com/vaadin/client/widgets/Grid.java b/client/src/com/vaadin/client/widgets/Grid.java index 2207abe408..28c26893ef 100644 --- a/client/src/com/vaadin/client/widgets/Grid.java +++ b/client/src/com/vaadin/client/widgets/Grid.java @@ -5688,14 +5688,8 @@ public class Grid<T> extends ResizeComposite implements if (SortDirection.ASCENDING == sortingOrder.getDirection()) { cellElement.addClassName("sort-asc"); - if (!sortedBefore) { - verifyColumnWidth(column); - } } else { cellElement.addClassName("sort-desc"); - if (!sortedBefore) { - verifyColumnWidth(column); - } } int sortIndex = Grid.this.getSortOrder().indexOf(sortingOrder); @@ -5705,6 +5699,10 @@ public class Grid<T> extends ResizeComposite implements cellElement.setAttribute("sort-order", String.valueOf(sortIndex + 1)); } + + if (!sortedBefore) { + verifyColumnWidth(column); + } } /** |