Added CSS class for sorted columns in adminhtml (OpenMage#2604)

Author: sreichel

app/code/core/Mage/Adminhtml/Block/Widget/Grid/Column.php

@@ -55,21 +55,33 @@ public function getGrid()
         return $this->_grid;
     }
 
+    /**
+     * @return bool
+     */
     public function isLast()
     {
         return $this->getId() == $this->getGrid()->getLastColumnId();
     }
 
+    /**
+     * @return string
+     */
     public function getHtmlProperty()
     {
         return $this->getRenderer()->renderProperty();
     }
 
+    /**
+     * @return string
+     */
     public function getHeaderHtml()
     {
         return $this->getRenderer()->renderHeader();
     }
 
+    /**
+     * @return string|null
+     */
     public function getCssClass()
     {
         if (is_null($this->_cssClass)) {
@@ -83,16 +95,26 @@ public function getCssClass()
             if ($this->getEditable()) {
                 $this->_cssClass .= ' editable';
             }
+            // Add css class for sorted columns
+            if ($this->hasData('dir')) {
+                $this->_cssClass .= ' sorted';
+            }
         }
 
         return $this->_cssClass;
     }
 
+    /**
+     * @return string|null
+     */
     public function getCssProperty()
     {
         return $this->getRenderer()->renderCss();
     }
 
+    /**
+     * @return string|null
+     */
     public function getHeaderCssClass()
     {
         $class = $this->getData('header_css_class');
@@ -102,9 +124,15 @@ public function getHeaderCssClass()
         if ($this->isLast()) {
             $class .= ' last';
         }
+        if ($this->hasData('dir')) {
+            $class .= ' sorted';
+        }
         return $class;
     }
 
+    /**
+     * @return string
+     */
     public function getHeaderHtmlProperty()
     {
         $str = '';
@@ -374,14 +402,16 @@ public function getFilter()
         return $this->_filter;
     }
 
+    /**
+     * @return string
+     */
     public function getFilterHtml()
     {
         if ($this->getFilter()) {
             return $this->getFilter()->getHtml();
-        } else {
-            return ' ';
         }
-        return null;
+
+        return ' ';
     }
 
     /**

js/mage/adminhtml/grid.js

@@ -55,6 +55,8 @@ varienGrid.prototype = {
             for (var row=0; row<this.rows.length; row++) {
                 if(row%2==0){
                     Element.addClassName(this.rows[row], 'even');
+                }else{
+                    Element.addClassName(this.rows[row], 'odd');
                 }
 
                 Event.observe(this.rows[row],'mouseover',this.trOnMouseOver);