Merge branch 'staging' into slide2

www/common/common-interface.js

@@ -58,35 +58,52 @@ define([
 
     var dialog = UI.dialog = {};
 
-    dialog.selectable = function (value) {
-        var input = h('input', {
+    var merge = function (a, b) {
+        var c = {};
+        if (a) {
+            Object.keys(a).forEach(function (k) {
+                c[k] = a[k];
+            });
+        }
+        if (b) {
+            Object.keys(b).forEach(function (k) {
+                c[k] = b[k];
+            });
+        }
+        return c;
+    };
+
+    dialog.selectable = function (value, opt) {
+        var attrs = merge({
             type: 'text',
             readonly: 'readonly',
-        });
+        }, opt);
+
+        var input = h('input', attrs);
         $(input).val(value).click(function () {
             input.select();
         });
         return input;
     };
 
-    dialog.okButton = function () {
-        return h('button.ok', { tabindex: '2', }, Messages.okButton);
+    dialog.okButton = function (content) {
+        return h('button.ok', { tabindex: '2', }, content || Messages.okButton);
     };
 
-    dialog.cancelButton = function () {
-        return h('button.cancel', { tabindex: '1'}, Messages.cancelButton);
+    dialog.cancelButton = function (content) {
+        return h('button.cancel', { tabindex: '1'}, content || Messages.cancelButton);
     };
 
     dialog.message = function (text) {
-        return h('p.message', text);
+        return h('p.msg', text);
     };
 
     dialog.textInput = function (opt) {
-        return h('input', opt || {
-            placeholder: '',
+        var attrs = merge({
             type: 'text',
             'class': 'cp-text-input',
-        });
+        }, opt);
+        return h('input', attrs);
     };
 
     dialog.nav = function (content) {
@@ -191,12 +208,20 @@ define([
 
     UI.alert = function (msg, cb, force) {
         cb = cb || function () {};
-        if (typeof(msg) === 'string' && force !== true) {
-            msg = Util.fixHTML(msg);
+
+        var message;
+        if (typeof(msg) === 'string') {
+            // sanitize
+            if (!force) { msg = Util.fixHTML(msg); }
+            message = dialog.message();
+            message.innerHTML = msg;
+        } else {
+            message = dialog.message(msg);
         }
+
         var ok = dialog.okButton();
         var frame = dialog.frame([
-            dialog.message(msg),
+            message,
             dialog.nav(ok),
         ]);
 
@@ -211,92 +236,102 @@ define([
         document.body.appendChild(frame);
         setTimeout(function () {
             $ok.focus();
-            if (typeof(UI.notify) === 'function') {
-                UI.notify();
-            }
+            UI.notify();
         });
     };
 
     UI.prompt = function (msg, def, cb, opt, force) {
-        opt = opt || {};
         cb = cb || function () {};
-        if (force !== true) { msg = Util.fixHTML(msg); }
+        opt = opt || {};
+
+        var input = dialog.textInput();
+        input.value = typeof(def) === 'string'? def: '';
+
+        var message;
+        if (typeof(msg) === 'string') {
+            if (!force) { msg = Util.fixHTML(msg); }
+            message = dialog.message();
+            message.innerHTML = msg;
+        } else {
+            message = dialog.message(msg);
+        }
+
+        var ok = dialog.okButton(opt.ok);
+        var cancel = dialog.cancelButton(opt.cancel);
+        var frame = dialog.frame([
+            message,
+            input,
+            dialog.nav([ cancel, ok, ]),
+        ]);
+
+        var listener;
+        var close = Util.once(function () {
+            $(frame).fadeOut(150, function () { $(this).remove(); });
+            stopListening(listener);
+        });
 
-        var keyHandler = listenForKeys(function () { // yes
-            findOKButton().click();
+        var $ok = $(ok).click(function (ev) { cb(input.value, ev); });
+        var $cancel = $(cancel).click(function (ev) { cb(null, ev); });
+        listener = listenForKeys(function () { // yes
+            close(); $ok.click();
         }, function () { // no
-            findCancelButton().click();
+            close(); $cancel.click();
         });
 
-        // Make sure we don't call both the "yes" and "no" handlers if we use "findOKButton().click()"
-        // in the callback
-        var isClicked = false;
-
-        Alertify
-            .defaultValue(def || '')
-            .okBtn(opt.ok || Messages.okButton || 'OK')
-            .cancelBtn(opt.cancel || Messages.cancelButton || 'Cancel')
-            .prompt(msg, function (val, ev) {
-                if (isClicked) { return; }
-                isClicked = true;
-                cb(val, ev);
-                stopListening(keyHandler);
-            }, function (ev) {
-                if (isClicked) { return; }
-                isClicked = true;
-                cb(null, ev);
-                stopListening(keyHandler);
-            });
-        if (typeof(UI.notify) === 'function') {
+        document.body.appendChild(frame);
+        setTimeout(function () {
+            input.select().focus();
             UI.notify();
-        }
+        });
     };
 
     UI.confirm = function (msg, cb, opt, force, styleCB) {
-        opt = opt || {};
         cb = cb || function () {};
-        if (force !== true) { msg = Util.fixHTML(msg); }
+        opt = opt || {};
 
-        var keyHandler = listenForKeys(function () {
-            findOKButton().click();
-        }, function () {
-            findCancelButton().click();
+        var message;
+        if (typeof(msg) === 'string') {
+            if (!force) { msg = Util.fixHTML(msg); }
+            message = dialog.message();
+            message.innerHTML = msg;
+        } else {
+            message = dialog.message(msg);
+        }
+
+        var ok = dialog.okButton(opt.ok);
+        var cancel = dialog.cancelButton(opt.cancel);
+
+        var frame = dialog.frame([
+            message,
+            dialog.nav(opt.reverseOrder?
+                [ok, cancel]: [cancel, ok]),
+        ]);
+
+        var listener;
+        var close = Util.once(function () {
+            $(frame).fadeOut(150, function () { $(this).remove(); });
+            stopListening(listener);
         });
 
-        // Make sure we don't call both the "yes" and "no" handlers if we use "findOKButton().click()"
-        // in the callback
-        var isClicked = false;
-
-        Alertify
-            .okBtn(opt.ok || Messages.okButton || 'OK')
-            .cancelBtn(opt.cancel || Messages.cancelButton || 'Cancel')
-            .confirm(msg, function () {
-                if (isClicked) { return; }
-                isClicked = true;
-                cb(true);
-                stopListening(keyHandler);
-            }, function () {
-                if (isClicked) { return; }
-                isClicked = true;
-                cb(false);
-                stopListening(keyHandler);
-            });
+        var $ok = $(ok).click(function (ev) { close(); cb(true, ev); });
+        var $cancel = $(cancel).click(function (ev) { close(); cb(false, ev); });
 
-        window.setTimeout(function () {
-            var $ok = findOKButton();
-            var $cancel = findCancelButton();
-            if (opt.okClass) { $ok.addClass(opt.okClass); }
-            if (opt.cancelClass) { $cancel.addClass(opt.cancelClass); }
-            if (opt.reverseOrder) {
-                $ok.insertBefore($ok.prev());
-            }
+        if (opt.cancelClass) { $cancel.addClass(opt.cancelClass); }
+        if (opt.okClass) { $ok.addClass(opt.okClass); }
+
+        listener = listenForKeys(function () {
+            $ok.click();
+        }, function () {
+            $cancel.click();
+        });
+
+        document.body.appendChild(frame);
+        setTimeout(function () {
+            UI.notify();
             if (typeof(styleCB) === 'function') {
                 styleCB($ok.closest('.dialog'));
             }
-        }, 0);
-        if (typeof(UI.notify) === 'function') {
-            UI.notify();
-        }
+        });
     };
 
     UI.log = function (msg) {

www/common/common-messenger.js

@@ -3,8 +3,7 @@ define([
     '/bower_components/chainpad-crypto/crypto.js',
     '/common/curve.js',
     '/common/common-hash.js',
-    '/common/common-realtime.js'
-], function ($, Crypto, Curve, Hash, Realtime) {
+], function ($, Crypto, Curve, Hash) {
     'use strict';
     var Msg = {
         inputs: [],
@@ -242,7 +241,7 @@ define([
             if (!proxy.friends) { return; }
             var friends = proxy.friends;
             delete friends[curvePublic];
-            Realtime.whenRealtimeSyncs(common, realtime, cb);
+            common.whenRealtimeSyncs(realtime, cb);
         };
 
         var pushMsg = function (channel, cryptMsg) {
@@ -472,7 +471,7 @@ define([
                 channel.wc.bcast(cryptMsg).then(function () {
                     delete friends[curvePublic];
                     delete channels[curvePublic];
-                    Realtime.whenRealtimeSyncs(common, realtime, function () {
+                    common.whenRealtimeSyncs(realtime, function () {
                         cb();
                     });
                 }, function (err) {

www/common/cryptpad-common.js

@@ -1089,6 +1089,7 @@ define([
     */
     var LIMIT_REFRESH_RATE = 30000; // milliseconds
     common.createUsageBar = function (cb) {
+        if (!isLoggedIn()) { return cb("NOT_LOGGED_IN"); }
         // getPinnedUsage updates common.account.usage, and other values
         // so we can just use those and only check for errors
         var $container = $('<span>', {'class':'limit-container'});

www/drive/main.js

@@ -2340,9 +2340,9 @@ define([
             $('<br>').appendTo($d);
             if (!ro) {
                 $('<label>', {'for': 'propLink'}).text(Messages.editShare).appendTo($d);
-                $('<input>', {'id': 'propLink', 'readonly': 'readonly', 'value': base + data.href})
-                .click(function () { $(this).select(); })
-                .appendTo($d);
+                $d.append(Cryptpad.dialog.selectable(base + data.href, {
+                    id: 'propLink',
+                }));
             }
 
             var parsed = Cryptpad.parsePadUrl(data.href);
@@ -2350,9 +2350,9 @@ define([
                 var roLink = ro ? base + data.href : getReadOnlyUrl(el);
                 if (roLink) {
                     $('<label>', {'for': 'propROLink'}).text(Messages.viewShare).appendTo($d);
-                    $('<input>', {'id': 'propROLink', 'readonly': 'readonly', 'value': roLink})
-                    .click(function () { $(this).select(); })
-                    .appendTo($d);
+                    $d.append(Cryptpad.dialog.selectable(roLink, {
+                        id: 'propROLink',
+                    }));
                 }
             }
 
@@ -2370,20 +2370,17 @@ define([
                         return void cb(void 0, $d);
                     }
                     var KB = Cryptpad.bytesToKilobytes(bytes);
+
+                    var formatted = Messages._getKey('formattedKB', [KB]);
                     $('<br>').appendTo($d);
 
                     $('<label>', {
                         'for': 'size'
                     }).text(Messages.fc_sizeInKilobytes).appendTo($d);
 
-                    $('<input>', {
+                    $d.append(Cryptpad.dialog.selectable(formatted, {
                         id: 'size',
-                        readonly: 'readonly',
-                        value: KB + 'KB',
-                    })
-                    .click(function () { $(this).select(); })
-                    .appendTo($d);
-
+                    }));
                     cb(void 0, $d);
                 });
             } else {
@@ -2439,8 +2436,7 @@ define([
                 var el = filesOp.find(paths[0].path);
                 getProperties(el, function (e, $prop) {
                     if (e) { return void logError(e); }
-                    Cryptpad.alert('', undefined, true);
-                    $('.alertify .msg').html("").append($prop);
+                    Cryptpad.alert($prop[0], undefined, true);
                 });
             }
             module.hideMenu();
@@ -2491,8 +2487,7 @@ define([
                 var el = filesOp.find(paths[0].path);
                 getProperties(el, function (e, $prop) {
                     if (e) { return void logError(e); }
-                    Cryptpad.alert('', undefined, true);
-                    $('.alertify .msg').html("").append($prop);
+                    Cryptpad.alert($prop[0], undefined, true);
                 });
             }
             module.hideMenu();

www/filepicker/inner.js

@@ -90,7 +90,7 @@ define([
 
             // Description
             var text = Messages.filePicker_description;
-            if (types.length === 1 && types[0] !== 'file') {
+            if (types && types.length === 1 && types[0] !== 'file') {
                 // Should be Templates
                 text = Messages.selectTemplate;
             }
@@ -117,7 +117,8 @@ define([
             // Update the files list when needed
             updateContainer = function () {
                 $container.html('');
-                var filter = $filter.find('.cp-filepicker-filter').val().trim();
+                var $input = $filter.find('.cp-filepicker-filter');
+                var filter = $input.val().trim();
                 var todo = function (err, list) {
                     if (err) { return void console.error(err); }
                     Object.keys(list).forEach(function (id) {
@@ -136,6 +137,7 @@ define([
                             if (typeof onSelect === "function") { onSelect(data.href); }
                         });
                     });
+                    $input.focus();
                 };
                 common.getFilesList(filters, todo);
             };

www/settings/main.js

@@ -75,9 +75,7 @@ define([
             var userHref = Cryptpad.getUserHrefFromKeys(accountName, publicKey);
             var $pubLabel = $('<span>', {'class': 'label'})
                 .text(Messages.settings_publicSigningKey);
-            var $pubKey = $('<input>', {type: 'text', readonly: true})
-                .val(userHref);
-            $key.append($pubLabel).append($pubKey);
+            $key.append($pubLabel).append(Cryptpad.dialog.selectable(userHref));
         }
 
         return $div;