- Fix wrong column sort order icons (#1485823)

- set defult sorting of new column to 'ASC'


git-svn-id: https://svn.roundcube.net/trunk@2433 208e9e7b-5314-0410-a742-e7e81cd9613c

1 files changed, 8 insertions, 13 deletions

diff --git a/roundcubemail/program/js/app.js b/roundcubemail/program/js/app.js
index 6293392cb..6080fba31 100644
--- a/roundcubemail/program/js/app.js
+++ b/roundcubemail/program/js/app.js
@@ -599,20 +599,15 @@ function rcube_webmail()
         // get the type of sorting
         var a_sort = props.split('_');
         var sort_col = a_sort[0];
-        var sort_order = a_sort[1] ? a_sort[1].toUpperCase() : null;
-
-        // no sort order specified: toggle
-        if (sort_order==null)
-          {
-          if (this.env.sort_col==sort_col)
-            sort_order = this.env.sort_order=='ASC' ? 'DESC' : 'ASC';
-          else
-            sort_order = this.env.sort_order;
-          }
-
-        if (this.env.sort_col==sort_col && this.env.sort_order==sort_order)
-          break;
+        var sort_order = a_sort[1] ? a_sort[1].toUpperCase() : 'ASC';
 
+        if (this.env.sort_col==sort_col) {
+          sort_order = this.env.sort_order=='ASC' ? 'DESC' : 'ASC';
+          
+	  if (this.env.sort_order==sort_order)
+            break;
+	}
+	
         // set table header class
         $('#rcm'+this.env.sort_col).removeClass('sorted'+(this.env.sort_order.toUpperCase()));
         $('#rcm'+sort_col).addClass('sorted'+sort_order);