BUGFIX: GridField filter

Make sure the new class is used consistently, and has the correct css applied to it.
silverstripe · May 16, 2012 · f0c61c2 · f0c61c2
1 parent eeef41e
commit f0c61c2
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/css/GridField.css b/css/GridField.css
diff --git a/forms/gridfield/GridFieldSortableHeader.php b/forms/gridfield/GridFieldSortableHeader.php
@@ -86,7 +86,7 @@ public function getHTMLFragments($gridField) {
 				}
 			} else {
 				if($currentColumn == count($columns) && $gridField->getConfig()->getComponentByType('GridFieldFilterHeader')){
-					$field = new LiteralField($columnField, '<button name="showFilter" class="ss-gridfield-button-filter" id="showFilter"></button>');				
+					$field = new LiteralField($columnField, '<button name="showFilter" class="ss-gridfield-button-filter trigger"></button>');				
 				}else{
 					$field = new LiteralField($columnField, '<span class="non-sortable">' . $title . '</span>');
 				}

diff --git a/javascript/GridField.js b/javascript/GridField.js
@@ -44,7 +44,7 @@
 						content = '<span class="non-sortable"></span>';
 						self.addClass('show-filter').find('.filter-header').show();														
 					}else{
-						content = '<button name="showFilter" class="ss-gridfield-button-filter"></button>';
+						content = '<button name="showFilter" class="ss-gridfield-button-filter trigger"></button>';
 						self.removeClass('show-filter').find('.filter-header').hide();	
 					}
 

diff --git a/scss/GridField.scss b/scss/GridField.scss
@@ -418,7 +418,7 @@ $gf_grid_x:	16px;
 								));		
 							} 	
 
-							&#showFilter{	//The magnifying glass before filter is opened						
+							&.trigger{	//The magnifying glass before filter is opened						
 								margin-left: $gf_grid_y; 							
 								border:none;						
 								@include background ($bgImage -17px 6px);