Ability to edit the title in the toolbar by clicking it

www/common/cryptpad-common.js

@@ -302,13 +302,17 @@ define([
         var type = parsed.type;
         var untitledIndex = 1;
         var name = (Messages.type)[type] + ' - ' + new Date().toString().split(' ').slice(0,4).join(' ');
-        if (isNameAvailable(name, parsed, recentPads)) { return name; }
-        while (!isNameAvailable(name + ' - ' + untitledIndex, parsed, recentPads)) { untitledIndex++; }
-        return name + ' - ' + untitledIndex;
+        return name;
+        /*
+         * Pad titles are shared in the document so it does not make sense anymore to avoid duplicates
+          if (isNameAvailable(name, parsed, recentPads)) { return name; }
+          while (!isNameAvailable(name + ' - ' + untitledIndex, parsed, recentPads)) { untitledIndex++; }
+          return name + ' - ' + untitledIndex;
+        */
     };
     var isDefaultName = common.isDefaultName = function (parsed, title) {
         var name = getDefaultName(parsed, []);
-        return title.slice(0, name.length) === name;
+        return title === name;
     };
 
     var makePad = function (href, title) {
@@ -622,7 +626,21 @@ define([
     var renamePad = common.renamePad = function (title, callback) {
         if (title === null) { return; }
 
-        common.causesNamingConflict(title, function (err, conflicts) {
+        if (title.trim() === "") {
+            var parsed = parsePadUrl(window.location.href);
+            title = getDefaultName(parsed);
+        }
+
+        common.setPadTitle(title, function (err, data) {
+            if (err) {
+                console.log("unable to set pad title");
+                console.log(err);
+                return;
+            }
+            callback(null, title);
+        });
+        /* Pad titles are shared in the document. We don't check for duplicates anymore.
+         common.causesNamingConflict(title, function (err, conflicts) {
             if (err) {
                 console.log("Unable to determine if name caused a conflict");
                 console.error(err);
@@ -644,6 +662,7 @@ define([
                 callback(null, title);
             });
         });
+        */
     };
     var createButton = common.createButton = function (type, rightside, data, callback) {
         var button;

www/common/toolbar.js

@@ -303,29 +303,40 @@ define([
         $(lagElement).append(lagLight);
     };
 
-    var createTitle = function ($container, readOnly, cb) {
+    var createTitle = function ($container, readOnly, config, Cryptpad) {
+        var callback = config.onRename;
+        var placeholder = config.defaultName;
+
         var $titleContainer = $('<span>', {
             id: 'toolbarTitle',
             'class': TITLE_CLS
         }).appendTo($container);
         var $text = $('<span>').appendTo($titleContainer);
-        if (readOnly === 1) { return; }
+        if (readOnly === 1 || typeof(Cryptpad) === "unedfined") { return; }
         var $input = $('<input>', {
-            type: 'text'
+            type: 'text',
+            placeholder: placeholder
         }).appendTo($titleContainer).hide();
+        $input.on('mousedown', function (e) {
+            if (!$input.is(":focus")) {
+                $input.focus();
+            }
+            e.stopPropagation();
+            return true;
+        });
         $input.on('keyup', function (e) {
             if (e.which === 13) {
-                Cryptpad.renamePad(title, function (err, newtitle) {
+                var name = $input.val().trim();
+                Cryptpad.renamePad($input.val(), function (err, newtitle) {
                     if (err) { return; }
                     $text.text(newtitle);
-                    cb(null, newtitle);
+                    callback(null, newtitle);
                     $input.hide();
                     $text.show();
                 });
             }
         });
         $text.on('click', function () {
-            console.log('click');
             $text.hide();
             $input.val($text.text());
             $input.show();
@@ -335,12 +346,13 @@ define([
 
     var create = Bar.create = function ($container, myUserName, realtime, getLag, userList, config) {
         var readOnly = (typeof config.readOnly !== "undefined") ? (config.readOnly ? 1 : 0) : -1;
+        var Cryptpad = config.common;
 
         var toolbar = createRealtimeToolbar($container);
         var userListElement = createUserList(toolbar.find('.' + LEFTSIDE_CLS), readOnly);
         var spinner = createSpinner(toolbar.find('.' + RIGHTSIDE_CLS));
         var lagElement = createLagElement(toolbar.find('.' + RIGHTSIDE_CLS));
-        var $titleElement = createTitle(toolbar.find('.' + TOP_CLS), readOnly, config.onRename);
+        var $titleElement = createTitle(toolbar.find('.' + TOP_CLS), readOnly, config.title, Cryptpad);
         var userData = config.userData;
         // readOnly = 1 (readOnly enabled), 0 (disabled), -1 (old pad without readOnly mode)
         var saveElement;