2872: Merge branch 'master' into 2872-folder-nav

[arvados.git] / apps / workbench / app / assets / javascripts / editable.js

diff --git a/apps/workbench/app/assets/javascripts/editable.js b/apps/workbench/app/assets/javascripts/editable.js
index a32eb7e548f06e6404d32586bb702484d89036f3..ab66833c286a6c4666c3162309fe8267be03fea9 100644 (file)
--- a/apps/workbench/app/assets/javascripts/editable.js
+++ b/apps/workbench/app/assets/javascripts/editable.js
@@ -41,10 +41,9 @@ $.fn.editable.defaults.validate = function (value) {
 
 $(document).
     on('ready ajax:complete', function() {
-        $('#editable-submit').click(function() {
-            console.log($(this));
-        });
         $('.editable').
+            not('.editable-done-setup').
+            addClass('editable-done-setup').
             editable({
                 success: function(response, newValue) {
                     // If we just created a new object, stash its UUID
@@ -57,6 +56,9 @@ $(document).
                         $(this).editable('option', 'url', response.href);
                     }
                     return;
+                },
+                error: function(response, newValue) {
+                    return response.responseJSON.errors.join();
                 }
             }).
             on('hidden', function(e, reason) {
@@ -77,10 +79,13 @@ $(document).
             });
     }).
     on('ready ajax:complete', function() {
-        $("[data-toggle~='x-editable']").click(function(e) {
-            e.stopPropagation();
-            $($(this).attr('data-toggle-selector')).editable('toggle');
-        });
+        $("[data-toggle~='x-editable']").
+            not('.editable-done-setup').
+            addClass('editable-done-setup').
+            click(function(e) {
+                e.stopPropagation();
+                $($(this).attr('data-toggle-selector')).editable('toggle');
+            });
     });
 
 $.fn.editabletypes.text.defaults.tpl = '<input type="text" name="editable-text">'