From c0415f43a5d50a1e67bc4fc26d65a53696f82844 Mon Sep 17 00:00:00 2001
From: aalpern <adam.alpern@gmail.com>
Date: Fri, 27 Mar 2015 11:52:37 -0700
Subject: [PATCH 1/3] Added some event debug logging

---
 js/edit/edit-mode.js | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/js/edit/edit-mode.js b/js/edit/edit-mode.js
index bb397fff..c2c7a105 100644
--- a/js/edit/edit-mode.js
+++ b/js/edit/edit-mode.js
@@ -1,11 +1,14 @@
 (function () {
 
+  var log = ds.log.logger('tessera.edit')
+
   /* -----------------------------------------------------------------------------
      Queries
      ----------------------------------------------------------------------------- */
 
   /* Query delete buttons */
   $(document).on('click', 'button.ds-delete-query-button', function(e) {
+    log.debug('click.delete-query')
     var $elt = $(this)
     var query_name   = $elt.attr('data-ds-query-name')
     var dashboard = ds.manager.current.dashboard
@@ -39,6 +42,7 @@
 
   /* Query duplicate buttons */
   $(document).on('click', 'button.ds-duplicate-query-button', function(e) {
+    log.debug('click.duplicate-query')
     var $elt       = $(this)
     var query_name = $elt.attr('data-ds-query-name')
     var dashboard  = ds.manager.current.dashboard
@@ -50,6 +54,7 @@
 
 
   ds.edit.edit_queries = function() {
+    log.debug('edit_queries()')
     /* Query names */
     $('th.ds-query-name').each(function(index, e) {
       var element = $(e)
@@ -85,6 +90,7 @@
    * Rename a query and update the UI to reflect the change.
    */
   function rename_query(dashboard, old_name, new_name) {
+    log.debug('rename_query()')
     var query = dashboard.definition.queries[old_name]
     var updated_items = dashboard.definition.rename_query(old_name, new_name)
     $('[data-ds-query-name="' + old_name + '"]').replaceWith(
@@ -103,6 +109,7 @@
    * Delete a query and remove it from the queries list in the UI.
    */
   function delete_query(dashboard, query_name) {
+    log.debug('delete_query()')
     dashboard.definition.delete_query(query_name)
     $('tr[data-ds-query-name="' + query_name + '"]').remove()
     ds.edit.edit_queries()
@@ -113,6 +120,7 @@
    * Add a new query to the dashboard and UI.
    */
   function add_query(dashboard, name, target) {
+    log.debug('add_query()')
     var query = ds.models.data.Query({name: name, targets: target})
     dashboard.definition.add_query(query)
     $("#ds-query-panel table").append(ds.templates.edit['dashboard-query-row'](query))
@@ -122,6 +130,7 @@
   }
 
   function duplicate_query(dashboard, name) {
+    log.debug('duplicate_query()')
     var new_name = 'Copy of ' + name + ' ' + Object.keys(dashboard.definition.queries).length
     var source   = dashboard.definition.queries[name]
     return add_query(dashboard, new_name, source.targets.slice(0))
@@ -134,6 +143,7 @@
    * be used as a placeholder.
    */
   function new_query(dashboard, targets) {
+    log.debug('new_query()')
     var name = "query" + Object.keys(dashboard.definition.queries).length
     return add_query(dashboard, name, targets || 'absolute(randomWalkFunction("' + name + '"))')
   }
@@ -232,13 +242,16 @@
 
   ds.app.add_mode_handler(ds.app.Mode.EDIT, {
     enter: function() {
+      log.debug('mode_handler.enter()')
       $('.ds-section, .ds-cell, .ds-row').addClass('ds-edit')
       ds.edit.edit_queries()
     },
     exit: function() {
+      log.debug('mode_handler.exit()')
       $('.ds-section, .ds-cell, .ds-row').removeClass('ds-edit')
     },
     refresh: function() {
+      log.debug('mode_handler.refresh()')
       $('.ds-section, .ds-cell, .ds-row').addClass('ds-edit')
     }
   })

From 6685e591e87e7974617bd5d14fbc09a0ede3f474 Mon Sep 17 00:00:00 2001
From: aalpern <adam.alpern@gmail.com>
Date: Fri, 27 Mar 2015 11:57:32 -0700
Subject: [PATCH 2/3] Add a tab event handler to make queries editable

Seems to be more reliable than doing it on mode enter
---
 js/edit/edit-mode.js               | 7 ++++++-
 templates/edit/dashboard_panel.hbs | 2 +-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/js/edit/edit-mode.js b/js/edit/edit-mode.js
index c2c7a105..cefa9b24 100644
--- a/js/edit/edit-mode.js
+++ b/js/edit/edit-mode.js
@@ -52,6 +52,12 @@
     return true
   })
 
+  $(document).on('shown.bs.tab', 'a[data-toggle="tab"]', function(e) {
+    log.debug('shown.bs.tab ' + e.target.href)
+    if (e.target.id == 'ds-edit-tab-queries') {
+      ds.edit.edit_queries()
+    }
+  })
 
   ds.edit.edit_queries = function() {
     log.debug('edit_queries()')
@@ -244,7 +250,6 @@
     enter: function() {
       log.debug('mode_handler.enter()')
       $('.ds-section, .ds-cell, .ds-row').addClass('ds-edit')
-      ds.edit.edit_queries()
     },
     exit: function() {
       log.debug('mode_handler.exit()')
diff --git a/templates/edit/dashboard_panel.hbs b/templates/edit/dashboard_panel.hbs
index 3fcd2e63..64e651f0 100644
--- a/templates/edit/dashboard_panel.hbs
+++ b/templates/edit/dashboard_panel.hbs
@@ -1,6 +1,6 @@
 <ul class="nav nav-pills">
   <li class="active"><a href="#metadata" data-toggle="tab">Metadata</a></li>
-  <li><a href="#queries" data-toggle="tab">Queries</a></li>
+  <li><a href="#queries" data-toggle="tab" id="ds-edit-tab-queries">Queries</a></li>
 </ul>
 
 <div class="tab-content">

From 62d3bae4e5e751071be8dd0a5636ded99bfd0fdc Mon Sep 17 00:00:00 2001
From: aalpern <adam.alpern@gmail.com>
Date: Fri, 27 Mar 2015 11:58:08 -0700
Subject: [PATCH 3/3] Version bump to 0.6.2

---
 tessera/_version.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tessera/_version.py b/tessera/_version.py
index b7519648..9c611f7e 100644
--- a/tessera/_version.py
+++ b/tessera/_version.py
@@ -1,2 +1,2 @@
-__version_info__ = (0, 6, 1)
+__version_info__ = (0, 6, 2)
 __version__ = '.'.join(map(str, __version_info__))