]> source.dussan.org Git - vaadin-framework.git/commitdiff
Verify Column width after all sort related cell changes (#16597)
authorTeemu Suo-Anttila <teemusa@vaadin.com>
Thu, 17 Dec 2015 13:56:43 +0000 (15:56 +0200)
committerVaadin Code Review <review@vaadin.com>
Fri, 18 Dec 2015 10:45:56 +0000 (10:45 +0000)
This fixes an IE8 issue where multisorting is not shown correctly.

Change-Id: Id68103f06baf9e3ae20cb2714407b467829ef1a0

client/src/com/vaadin/client/widgets/Grid.java

index 2207abe4080aec3dcf62a4f7346792887fa963a8..28c26893ef6324aabea7acc016976d0a81165375 100644 (file)
@@ -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);
+            }
         }
 
         /**