Make the first column look sorted by default, and a little clean-up.

2 files changed, 9 insertions, 7 deletions

diff --git a/CHANGES.txt b/CHANGES.txt
index 4fd9526..4320c70 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -5,7 +5,9 @@ Change history for Coverage.py
 Version 3.2b4

 -------------

 

-- None yet!

+- Click-to-sort HTML report columns are now persisted in a cookie.  Viewing

+  a report will sort it first the way you last had a coverage report sorted.

+  Thanks, `Chris Adams`_.

 

 

 Version 3.2b3, 23 November 2009

diff --git a/coverage/htmlfiles/index.html b/coverage/htmlfiles/index.html
index cd5a27b..4443349 100644
--- a/coverage/htmlfiles/index.html
+++ b/coverage/htmlfiles/index.html
@@ -22,7 +22,7 @@
                 <thead>

                     {# The title='' attr doesn't work in Safari. #}

                     <tr class='tablehead' title='Click to sort'>

-                        <th class='name left'>Module</th>

+                        <th class='name left headerSortDown'>Module</th>

                         <th>statements</th>

                         <th>run</th>

                         <th>excluded</th>

@@ -103,7 +103,8 @@
                             // This table hasn't been sorted before - we'll use

                             // our stored settings:

                             jQuery(table).trigger('sorton', [sort_list]);

-                        } else {

+                        }

+                        else {

                             // This is not the first load - something has

                             // already defined sorting so we'll just update         

                             // our stored value to match:

@@ -117,11 +118,10 @@
                 var headers = {};

                 var col_count = jQuery("table.index > thead > tr > th").length;

 

-                for (var i = 0; i < col_count; i++) {

-                    headers[i] = {sorter: "digit"};

+                headers[0] = { sorter: 'text' };

+                for (var i = 1; i < col_count-1; i++) {

+                    headers[i] = { sorter: 'digit' };

                 }

-

-                headers[0]           = { sorter:'text' };

                 headers[col_count-1] = { sorter:'percent' };

 

                 // Enable the table sorter: