Fix #418 - header menu icons not visible in FF when used with sortabl…

…e columns
mleibman · May 20, 2013 · db5d6d6 · db5d6d6
1 parent c73424c
commit db5d6d6
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/examples/example-plugin-headerbuttons.html b/examples/example-plugin-headerbuttons.html
@@ -48,7 +48,7 @@
   var grid;
   var data = [];
   var options = {
-    enableCellNavigation:true
+    enableCellNavigation: true
   };
   var columns = [];
   var columnsWithHighlightingById = {};
@@ -61,6 +61,7 @@
       name: String.fromCharCode("A".charCodeAt(0) + i),
       field: i,
       width: 90,
+      sortable: true,
       formatter: highlightingFormatter,
       header: {
         buttons: [

diff --git a/plugins/slick.headerbuttons.css b/plugins/slick.headerbuttons.css
@@ -1,4 +1,5 @@
-.slick-column-name {
+.slick-column-name,
+.slick-sort-indicator {
   /**
    * This makes all "float:right" elements after it that spill over to the next line
    * display way below the lower boundary of the column thus hiding them.

diff --git a/slick.grid.css b/slick.grid.css
@@ -48,6 +48,8 @@ classes should alter those!
   width: 8px;
   height: 5px;
   margin-left: 4px;
+  margin-top: 6px;
+  float: left;
 }
 
 .slick-sort-indicator-desc {