mitigated annotation sqishing w/ min-width
authorDan Vanderkam <danvdk@gmail.com>
Tue, 2 Jul 2013 03:01:48 +0000 (23:01 -0400)
committerDan Vanderkam <danvdk@gmail.com>
Tue, 2 Jul 2013 03:01:48 +0000 (23:01 -0400)
extras/super-annotations.js
tests/hairlines.html

index efc0aac..a63d8eb 100644 (file)
@@ -97,6 +97,7 @@ annotations.prototype.makeAnnotationEditable = function(a) {
   a.editable = true;
   var editableTemplateDiv = $('#annotation-editable-template').get(0);
   a.infoDiv.innerHTML = this.getTemplateHTML(editableTemplateDiv, a);
+  $(a.infoDiv).toggleClass('editable', !!a.editable);
   $(this).triggerHandler('beganEditAnnotation', a);
 };
 
@@ -284,6 +285,7 @@ annotations.prototype.updateAnnotationInfo = function() {
   $.each(this.annotations_, function(idx, a) {
     // We should never update an editable div -- doing so may kill unsaved
     // edits to an annotation.
+    $(a.infoDiv).toggleClass('editable', !!a.editable);
     if (a.editable) return;
     a.infoDiv.innerHTML = that.getTemplateHTML(templateDiv, a);
   });
index 8318602..fcdf3f8 100644 (file)
         display: table;  /* shrink to fit */
         box-shadow: 0 0 4px gray;
         cursor: move;
+
+        min-width: 120px;  /* prevents squishing at the right edge of the chart */
+      }
+      .annotation-info.editable {
+        min-width: 180px;  /* prevents squishing at the right edge of the chart */
       }
 
       .dygraph-annotation-line {