fix stats links, also tooltip headers and truncate

mozilla · Oct 27, 2011 · 13fa9d9 · 13fa9d9
1 parent 4b57cca
commit 13fa9d9
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 12 deletions.
diff --git a/apps/stats/helpers.py b/apps/stats/helpers.py
@@ -15,12 +15,12 @@ def report_menu(context, request, addon, report):
     report_tree = [
         {
             'name': 'overview',
-            'url': '/',
+            'url': '',
             'title': _('Overview'),
         },
         {
             'name': 'downloads',
-            'url': '/downloads/',
+            'url': 'downloads/',
             'title': _('Downloads'),
             'children': [
                 {
@@ -32,32 +32,32 @@ def report_menu(context, request, addon, report):
         },
         {
             'name': 'usage',
-            'url': '/usage/',
+            'url': 'usage/',
             'title': _('Daily Users'),
             'children': [
                 {
                     'name': 'versions',
-                    'url': '/usage/versions/',
+                    'url': 'usage/versions/',
                     'title': _('by Add-on Version')
                 },
                 {
                     'name': 'apps',
-                    'url': '/usage/applications/',
+                    'url': 'usage/applications/',
                     'title': _('by Application')
                 },
                 {
                     'name': 'locales',
-                    'url': '/usage/languages/',
+                    'url': 'usage/languages/',
                     'title': _('by Language')
                 },
                 {
                     'name': 'os',
-                    'url': '/usage/os/',
+                    'url': 'usage/os/',
                     'title': _('by Platform')
                 },
                 {
                     'name': 'statuses',
-                    'url': '/usage/status/',
+                    'url': 'usage/status/',
                     'title': _('by Add-on Status')
                 },
             ]
@@ -69,7 +69,7 @@ def report_menu(context, request, addon, report):
             addon.has_author(request.amo_user))):
         report_tree.append({
             'name': 'contributions',
-            'url': '/contributions/',
+            'url': 'contributions/',
             'title': _('Contributions')
         })
 

diff --git a/media/css/impala/stats.less b/media/css/impala/stats.less
@@ -211,6 +211,8 @@ table tbody tr {
     }
     thead th {
         white-space: normal;
+        overflow: hidden;
+        text-overflow: ellipsis;
     }
     th, td {
         width: 100px;

diff --git a/media/js/impala/stats/table.js b/media/js/impala/stats/table.js
@@ -75,9 +75,10 @@
                          row;
 
                      _.each(fields, function(f) {
-                         newHead += '<th>';
-                         var id = f.split('|').pop();
-                         newHead += z.StatsManager.getPrettyName(metric, id);
+                         var id = f.split('|').pop(),
+                             prettyName = z.StatsManager.getPrettyName(metric, id);
+                         newHead += format('<th title="{0}">', prettyName);
+                         newHead += prettyName;
                          newHead += '</th>';
                      });