From: Tim Bunce <Tim.Bunce@pobox.com>
Date: Thu, 20 Sep 2012 09:32:53 +0000 (+0100)
Subject: smarten up the tooltip
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=5ab994e65f7553e26a754ad3aaf3cbc573eaff18;p=p5sagit%2FDevel-Size.git

smarten up the tooltip
---

diff --git a/static/public/.tm.js.swp b/static/public/.tm.js.swp
index 2564ba6..2fb70b7 100644
Binary files a/static/public/.tm.js.swp and b/static/public/.tm.js.swp differ
diff --git a/static/public/css/Treemap.css b/static/public/css/Treemap.css
index c4b3d02..5e65db8 100644
--- a/static/public/css/Treemap.css
+++ b/static/public/css/Treemap.css
@@ -26,12 +26,12 @@
 /*TOOLTIPS*/
 .tip {
     color: #fff;
-    width: 139px;
+    width: auto;
     background-color: black;
     opacity:0.9;
     filter:alpha(opacity=90);
     font-size:10px;
-    font-family:Verdana, Geneva, Arial, Helvetica, sans-serif;
+    font-family: Menlo, Monaco, "Courier New", monospace;
     padding:7px;
 }
 
diff --git a/static/public/css/base.css b/static/public/css/base.css
index dafd35e..1b08135 100644
--- a/static/public/css/base.css
+++ b/static/public/css/base.css
@@ -109,6 +109,8 @@ a {
 .tip {
     color: #111;
     width: 139px;
+    text-align: left;
+    white-space: pre;
     background-color: white;
     border:1px solid #ccc;
     -moz-box-shadow:#555 2px 2px 8px;
@@ -120,4 +122,4 @@ a {
     font-size:10px;
     font-family:Verdana, Geneva, Arial, Helvetica, sans-serif;
     padding:7px;
-}
\ No newline at end of file
+}
diff --git a/static/public/tm.js b/static/public/tm.js
index bd689cd..6576b3a 100644
--- a/static/public/tm.js
+++ b/static/public/tm.js
@@ -25,6 +25,25 @@ var Log = {
   }
 };
 
+// http://stackoverflow.com/questions/5199901/how-to-sort-an-associative-array-by-its-values-in-javascript
+function bySortedValue(obj, comparitor, callback, context) {
+    var tuples = [];
+    for (var key in obj) {
+        if (obj.hasOwnProperty(key)) {
+            tuples.push([key, obj[key]]);
+        }
+    }
+
+    tuples.sort(comparitor);
+
+    if (callback) {
+        var length = tuples.length;
+        while (length--) callback.call(context, tuples[length][0], tuples[length][1]);
+    }
+    return tuples;
+}
+
+
 
 function init(){
   //init data
@@ -100,15 +119,31 @@ function init(){
           + "</div><div class=\"tip-text\">";
         var data = node.data;
 
+        html += "<br />";
         html += sprintf("Size: %d (%d + %d)<br />", data.self_size+data.kids_size, data.self_size, data.kids_size);
+
+        if (data.self_size) {
+            html += sprintf("Memory usage:<br />");
+            bySortedValue(data.leaves,
+                function(a, b) { return a[1] - b[1] },
+                function(k, v) { html += sprintf(" %10s: %5d<br />", k, v);
+            });
+            html += "<br />";
+        }
+
+        html += sprintf("Attributes:<br />");
+        bySortedValue(data.attr,
+            function(a, b) { return a[0] > b[0] ? 1 : a[0] < b[0] ? -1 : 0 },
+            function(k, v) { html += sprintf(" %10s: %5d<br />", k, v);
+        });
+        html += "<br />";
+
         if (data.child_count) {
             html += sprintf("Children: %d of %d<br />", data.child_count, data.kids_node_count);
         }
+        html += sprintf("Id: %s%s<br />", node.id, data._ids_merged ? data._ids_merged : "");
         html += sprintf("Depth: %d<br />", data.depth);
         html += sprintf("Parent: %d<br />", data.parent_id);
-        html += sprintf("Id: %s%s<br />", node.id, data._ids_merged ? data._ids_merged : "");
-        html += JSON.stringify(data.attr, undefined, 4) + "<br />";
-        html += JSON.stringify(data.leaves, undefined, 4) + "<br />";
 
         tip.innerHTML =  html; 
       }