Merge branch 'newtoolbar' into hpbs

customize.dist/main.css

@@ -1127,6 +1127,29 @@ html.cp,
 .cp div.realtime #addoption {
   border-bottom-left-radius: 5px;
 }
+.cp.slide #modal .button {
+  position: absolute;
+  cursor: pointer;
+  font-size: 30px;
+  opacity: 0.6;
+  display: none;
+}
+.cp.slide #modal .button:hover {
+  opacity: 1;
+  display: block !important;
+}
+.cp.slide #modal #button_exit {
+  left: 20px;
+  top: 20px;
+}
+.cp.slide #modal #button_left {
+  left: 6vw;
+  bottom: 10vh;
+}
+.cp.slide #modal #button_right {
+  right: 6vw;
+  bottom: 10vh;
+}
 .cp.slide #modal #content p,
 .cp.slide #modal #content ul,
 .cp.slide #modal #content ol {

customize.dist/src/less/cryptpad.less

@@ -722,6 +722,31 @@ form.realtime, div.realtime {
 }
 
 &.slide {
+    #modal {
+        .button {
+            position: absolute;
+            cursor: pointer;
+            font-size: 30px;
+            opacity: 0.6;
+            display: none;
+        }
+        .button:hover {
+            opacity: 1;
+            display: block !important;
+        }
+        #button_exit {
+            left: 20px;
+            top: 20px;
+        }
+        #button_left {
+            left: 6vw;
+            bottom: 10vh;
+        }
+        #button_right {
+            right: 6vw;
+            bottom: 10vh;
+        }
+    }
     #modal #content {
         p, ul, ol { font-size: 26px; }
 

www/code/main.js

@@ -684,8 +684,9 @@ define([
                         module.patchText(shjson2);
                     }
                 }
-
+                if (oldDoc !== remoteDoc) {
                     notify();
+                }
             };
 
             var onAbort = config.onAbort = function (info) {

www/pad/main.js

@@ -477,6 +477,8 @@ define([
             var onRemote = realtimeOptions.onRemote = function (info) {
                 if (initializing) { return; }
 
+                var oldShjson = stringifyDOM(inner);
+
                 var shjson = info.realtime.getUserDoc();
 
                 // remember where the cursor is
@@ -485,6 +487,11 @@ define([
                 // Update the user list (metadata) from the hyperjson
                 updateMetadata(shjson);
 
+                var newInner = JSON.parse(shjson);
+                if (newInner.length > 2) {
+                    var newSInner = stringify(newInner[2]);
+                }
+
                 // build a dom from HJSON, diff, and patch the editor
                 applyHjson(shjson);
 
@@ -518,7 +525,12 @@ define([
                         }
                     }
                 }
+
+                // Notify only when the content has changed, not when someone has joined/left
+                var oldSInner = stringify(JSON.parse(oldShjson)[2]);
+                if (newSInner !== oldSInner) {
                     notify();
+                }
             };
 
             var getHTML = function (Dom) {

www/poll/index.html

@@ -4,6 +4,11 @@
     <meta content="text/html; charset=utf-8" http-equiv="content-type"/>
     <meta name="viewport" content="width=device-width, initial-scale=1.0"/>
     <title data-localization="poll_title">Zero Knowledge Date Picker</title>
+    <link rel="icon" type="image/png"
+        href="/customize/main-favicon.png"
+        data-main-favicon="/customize/main-favicon.png"
+        data-alt-favicon="/customize/alt-favicon.png"
+        id="favicon" />
     <link rel="stylesheet" href="/bower_components/components-font-awesome/css/font-awesome.min.css">
     <link rel="stylesheet" href="/customize/main.css" />
     <script data-main="main" src="/bower_components/requirejs/require.js"></script>

www/poll/main.js

@@ -12,7 +12,7 @@ define([
     '/common/notify.js',
     '/bower_components/file-saver/FileSaver.min.js',
     '/bower_components/jquery/dist/jquery.min.js',
-], function (Config, Messages, TextPatcher, Listmap, Crypto, Cryptpad, Hyperjson, Renderer, Toolbar) {
+], function (Config, Messages, TextPatcher, Listmap, Crypto, Cryptpad, Hyperjson, Renderer, Toolbar, Visible, Notify) {
     var $ = window.jQuery;
 
     var unlockHTML = '<i class="fa fa-unlock" aria-hidden="true"></i>';
@@ -187,6 +187,9 @@ define([
 
     /*  Any time the realtime object changes, call this function */
     var change = function (o, n, path, throttle) {
+        if (!Cryptpad.isArray(path)) {
+            return;
+        }
         if (path && path.join) {
             console.log("Change from [%s] to [%s] at [%s]",
                 o, n, path.join(', '));
@@ -210,6 +213,7 @@ define([
 
             https://developer.mozilla.org/en-US/docs/Web/Security/Securing_your_site/Turning_off_form_autocompletion
         */
+        notify();
 
         if (throttle) {
             if (APP.throttled) { window.clearTimeout(APP.throttled); }
@@ -391,6 +395,20 @@ define([
         });
     };
 
+    var unnotify = function () {
+        if (APP.tabNotification &&
+            typeof(APP.tabNotification.cancel) === 'function') {
+            APP.tabNotification.cancel();
+        }
+    };
+
+    var notify = function () {
+        if (Visible.isSupported() && !Visible.currently()) {
+            unnotify();
+            APP.tabNotification = Notify.tab(1000, 10);
+        }
+    };
+
     var userData = APP.userData = {}; // List of pretty names for all users (mapped with their ID)
     var userList; // List of users still connected to the channel (server IDs)
     var addToUserData = function(data) {
@@ -575,6 +593,7 @@ define([
             .on('change', ['info'], function (o, n, p) {
                 if (p[1] === 'title') {
                     updateTitle(n);
+                    notify();
                 } else if (p[1] === "userData") {
                     addToUserData(APP.proxy.info.userData);
                 } else if (p[1] === 'description') {
@@ -591,6 +610,7 @@ define([
                         el.selectionStart = selects[0];
                         el.selectionEnd = selects[1];
                     }
+                    notify();
                 }
 
                 console.log("change: (%s, %s, [%s])", o, n, p.join(', '));
@@ -600,6 +620,13 @@ define([
 
         addToUserData(APP.proxy.info.userData);
 
+        if (Visible.isSupported()) {
+            Visible.onChange(function (yes) {
+                if (yes) { unnotify(); }
+            });
+        }
+
+
         getLastName(function (err, lastName) {
             APP.ready = true;
 

www/slide/inner.html

@@ -92,6 +92,9 @@
 
     <span class="cp slide">
     <div id="modal">
+        <div id="button_exit" class="button"><span class="fa fa-times"></span></div>
+        <div id="button_left" class="button"><span class="fa fa-chevron-left"></span></div>
+        <div id="button_right" class="button"><span class="fa fa-chevron-right"></span></div>
         <div id="content"></div>
     </div>
     </span>

www/slide/main.js

@@ -768,7 +768,9 @@ define([
                 }
                 Slide.update(remoteDoc);
 
+                if (oldDoc !== newDoc) {
                     notify();
+                }
             };
 
             var onAbort = config.onAbort = function (info) {

www/slide/slide.js

@@ -202,6 +202,31 @@ define([
     };
 
     var addEvent = function () {
+        var icon_to;
+        $modal.mousemove(function (e) {
+            var $buttons = $modal.find('.button');
+            $buttons.show();
+            if (icon_to) { window.clearTimeout(icon_to); }
+            icon_to = window.setTimeout(function() {
+                $buttons.fadeOut();
+            }, 1000);
+        });
+        $modal.find('#button_exit').click(function (e) {
+            var e = jQuery.Event("keyup");
+            e.which = 27;
+            $modal.trigger(e);
+        });
+        $modal.find('#button_left').click(function (e) {
+            var e = jQuery.Event("keyup");
+            e.which = 37;
+            $modal.trigger(e);
+        });
+        $modal.find('#button_right').click(function (e) {
+            var e = jQuery.Event("keyup");
+            e.which = 39;
+            $modal.trigger(e);
+        });
+
         $(ifrw).on('keyup', function (e) {
             if (!Slide.shown) { return; }
             switch(e.which) {