Visual editor: editing meta data via contenteditable instead of inputs

diff --git a/editor.css b/editor.css
index 651b58b..58ed300 100644 (file)
--- a/editor.css
+++ b/editor.css
@@ -60,17 +60,18 @@ body {
     float:right;\r
 }\r
 \r
-#rng-visualEditor-meta table input {\r
+#rng-visualEditor-meta table [contenteditable] {\r
     font-size: 11px;\r
     line-height:13px;\r
+    cursor: pointer;\r
 \r
 }\r
 \r
-#rng-visualEditor-meta table tr td:nth-child(1) input  {\r
+#rng-visualEditor-meta table tr td:nth-child(1) [contenteditable] {\r
     width:90px;\r
 }\r
 \r
-#rng-visualEditor-meta table tr td:nth-child(2) input  {\r
+#rng-visualEditor-meta table tr td:nth-child(2) [contenteditable]  {\r
     width:150px;\r
 }\r
 \r

diff --git a/modules/visualEditor.js b/modules/visualEditor.js
index 5833b71..dc0b801 100644 (file)
--- a/modules/visualEditor.js
+++ b/modules/visualEditor.js
@@ -21,7 +21,7 @@ rng.modules.visualEditor = function(sandbox) {
                 view._markSelected($(e.target));\r
             });\r
 \r
-            this.node.on('keyup', function(e) {\r
+            this.node.on('keyup', '#rng-visualEditor-contentWrapper', function(e) {\r
                 var anchor = $(window.getSelection().anchorNode);\r
                 if(anchor[0].nodeType === Node.TEXT_NODE)\r
                     anchor = anchor.parent();\r
@@ -31,10 +31,11 @@ rng.modules.visualEditor = function(sandbox) {
             });\r
             \r
             \r
-            this.metaTable = this.node.find('#rng-visualEditor-meta table');\r
+            var metaTable = this.metaTable = this.node.find('#rng-visualEditor-meta table');\r
             \r
             this.metaTable.find('.rng-visualEditor-metaAddBtn').click(function() {\r
-                view._addMetaRow();\r
+                var newRow = view._addMetaRow('', '');\r
+                $(newRow.find('td div')[0]).focus();\r
                 isDirty = true;\r
             });\r
             \r
@@ -42,15 +43,31 @@ rng.modules.visualEditor = function(sandbox) {
                 $(e.target).closest('tr').remove();\r
                 isDirty = true;\r
             });\r
+            \r
+            this.metaTable.on('keydown', '[contenteditable]', function(e) {\r
+                console.log(e.which);\r
+                if(e.which === 13) { \r
+                    if($('*:focus').hasClass('rng-visualEditor-metaItemKey')) {\r
+                        metaTable.find('.rng-visualEditor-metaItemValue').focus();\r
+                    } else {\r
+                        var input = $('<input>');\r
+                        input.appendTo('body').focus()\r
+                        metaTable.find('.rng-visualEditor-metaAddBtn').focus();\r
+                        input.remove();\r
+                    }\r
+                    e.preventDefault();\r
+                }\r
+                \r
+            });\r
 
         },\r
         getMetaData: function() {\r
             var toret = {};\r
             this.metaTable.find('tr').not('.rng-visualEditor-addMetaRow').each(function() {\r
                 var tr = $(this);\r
-                var inputs = $(this).find('td input');\r
-                var key = $(inputs[0]).val()\r
-                var value = $(inputs[1]).val()\r
+                var inputs = $(this).find('td [contenteditable]');\r
+                var key = $(inputs[0]).text();\r
+                var value = $(inputs[1]).text();\r
                 toret[key] = value;\r
             });\r
             console.log(toret);\r
@@ -75,7 +92,9 @@ rng.modules.visualEditor = function(sandbox) {
         },\r
         _addMetaRow: function(key, value) {\r
             var addRow = this.metaTable.find('.rng-visualEditor-addMetaRow');\r
-            $(sandbox.getTemplate('metaItem')({key: key || '', value: value || ''})).insertBefore(addRow);\r
+            var newRow = $(sandbox.getTemplate('metaItem')({key: key || '', value: value || ''}));\r
+            newRow.insertBefore(addRow);\r
+            return newRow;\r
         }\r
     };\r
     view.setup();\r