From efdae19f0cda9b3579fa2fdbb716be3cb237a453 Mon Sep 17 00:00:00 2001
From: yflory <yann.flory@xwiki.com>
Date: Thu, 19 Jan 2017 12:13:57 +0100
Subject: [PATCH 1/4] Update the lists in the code and slide toolbars to use
 the new dropdown

---
 customize.dist/main.css               |  2 +
 customize.dist/src/less/dropdown.less |  2 +
 customize.dist/toolbar.css            |  2 +
 www/code/main.js                      | 95 +++++++++++++++------------
 www/slide/main.js                     | 58 ++++++++--------
 5 files changed, 87 insertions(+), 72 deletions(-)

diff --git a/customize.dist/main.css b/customize.dist/main.css
index 157346865..0abf7e6bf 100644
--- a/customize.dist/main.css
+++ b/customize.dist/main.css
@@ -347,6 +347,8 @@
   min-width: 160px;
   box-shadow: 0px 1px 5px 0px rgba(0, 0, 0, 0.2);
   z-index: 1000;
+  max-height: 300px;
+  overflow-y: auto;
 }
 .dropdown-bar .dropdown-bar-content.left {
   right: 0;
diff --git a/customize.dist/src/less/dropdown.less b/customize.dist/src/less/dropdown.less
index d45749fa3..80fb08bff 100644
--- a/customize.dist/src/less/dropdown.less
+++ b/customize.dist/src/less/dropdown.less
@@ -29,6 +29,8 @@
         min-width: 160px;
         box-shadow: 0px 1px 5px 0px rgba(0, 0, 0, 0.2);
         z-index: 1000;
+        max-height: 300px;
+        overflow-y: auto;
 
         &.left {
             right: 0;
diff --git a/customize.dist/toolbar.css b/customize.dist/toolbar.css
index ab02c7b41..a169fb4ce 100644
--- a/customize.dist/toolbar.css
+++ b/customize.dist/toolbar.css
@@ -16,6 +16,8 @@
   min-width: 160px;
   box-shadow: 0px 1px 5px 0px rgba(0, 0, 0, 0.2);
   z-index: 1000;
+  max-height: 300px;
+  overflow-y: auto;
 }
 .dropdown-bar .dropdown-bar-content.left {
   right: 0;
diff --git a/www/code/main.js b/www/code/main.js
index 47ac9d919..3af11316d 100644
--- a/www/code/main.js
+++ b/www/code/main.js
@@ -77,7 +77,10 @@ define([
                 }
                 CodeMirror.autoLoadMode(editor, mode);
                 editor.setOption('mode', mode);
-                if ($select && $select.val) { $select.val(mode); }
+                if ($select) {
+                    var name = $select.find('a[data-value="' + mode + '"]').text() || 'Mode';
+                    $select.find('.buttonTitle').text(name);
+                }
             };
 
             editor.setValue(Messages.codeInitialState); // HERE
@@ -107,7 +110,7 @@ define([
                         }
                         editor.setOption('theme', theme);
                     }
-                    if ($select && $select.val) { $select.val(theme || 'default'); }
+                    if ($select) { $select.find('.buttonTitle').text(theme || 'Theme'); }
                 };
             }());
 
@@ -441,58 +444,68 @@ define([
 
                 var configureLanguage = function (cb) {
                     // FIXME this is async so make it happen as early as possible
-
-                    /*  Let the user select different syntax highlighting modes */
-                    var $language = module.$language = $('<select>', {
-                        title: 'syntax highlighting',
-                        id: 'language-mode',
-                        'class': 'rightside-element'
-                    }).on('change', function () {
-                        setMode($language.val());
-                        onLocal();
+                    var options = [];
+                    Modes.list.forEach(function (l) {
+                        options.push({
+                            tag: 'a',
+                            attributes: {
+                                'data-value': l.mode,
+                                'href': '#',
+                            },
+                            content: l.language // Pretty name of the language value
+                        });
                     });
-
-                    Modes.list.map(function (o) {
-                        $language.append($('<option>', {
-                            value: o.mode,
-                        }).text(o.language));
+                    var dropdownConfig = {
+                        text: 'Mode', // Button initial text
+                        options: options, // Entries displayed in the menu
+                        left: true, // Open to the left of the button
+                    };
+                    var $block = module.$language = Cryptpad.createDropdown(dropdownConfig);
+                    var $button = $block.find('.buttonTitle');
+
+                    $block.find('a').click(function (e) {
+                        setMode($(this).attr('data-value'));
+                        $button.text($(this).text());
                     });
-                    $rightside.append($language);
+
+                    $rightside.append($block);
                     cb();
                 };
 
-
                 var configureTheme = function () {
                     /*  Remember the user's last choice of theme using localStorage */
                     var themeKey = 'CRYPTPAD_CODE_THEME';
                     var lastTheme = localStorage.getItem(themeKey) || 'default';
 
-                    /*  Let the user select different themes */
-                    var $themeDropdown = $('<select>', {
-                        title: 'color theme',
-                        id: 'display-theme',
-                        'class': 'rightside-element'
-                    });
-                    Themes.forEach(function (o) {
-                        $themeDropdown.append($('<option>', {
-                            selected: o.name === lastTheme,
-                        }).val(o.name).text(o.name));
+                    var options = [];
+                    Themes.forEach(function (l) {
+                        options.push({
+                            tag: 'a',
+                            attributes: {
+                                'data-value': l.name,
+                                'href': '#',
+                            },
+                            content: l.name // Pretty name of the language value
+                        });
                     });
-
-
-                    $rightside.append($themeDropdown);
-
-                    var $theme = $bar.find('select#display-theme');
-
-                    setTheme(lastTheme, $theme);
-
-                    $theme.on('change', function () {
-                        var theme = $theme.val();
-                        console.log("Setting theme to %s", theme);
-                        setTheme(theme, $theme);
-                        // remember user choices
+                    var dropdownConfig = {
+                        text: 'Theme', // Button initial text
+                        options: options, // Entries displayed in the menu
+                        left: true, // Open to the left of the button
+                    };
+                    var $block = module.$theme = Cryptpad.createDropdown(dropdownConfig);
+                    var $button = $block.find('.buttonTitle');
+
+                    setTheme(lastTheme, $block);
+
+                    $block.find('a').click(function (e) {
+                        var theme = $(this).attr('data-value');
+                        setTheme(theme, $block);
+                        $button.text($(this).text());
                         localStorage.setItem(themeKey, theme);
                     });
+
+                    $rightside.append($block);
                 };
 
                 if (!readOnly) {
diff --git a/www/slide/main.js b/www/slide/main.js
index aecc139b1..a44f11c85 100644
--- a/www/slide/main.js
+++ b/www/slide/main.js
@@ -126,7 +126,7 @@ define([
                         }
                         editor.setOption('theme', theme);
                     }
-                    if ($select && $select.val) { $select.val(theme || 'default'); }
+                    if ($select) { $select.find('.buttonTitle').text(theme || 'Theme'); }
                 };
             }());
 
@@ -499,44 +499,40 @@ define([
                 }
                 $rightside.append($leavePresent);
 
-
                 var configureTheme = function () {
-                    /*var $language = $('<span>', {
-                        'style': "margin-right: 10px;",
-                        'class': 'rightside-element'
-                    }).text("Markdown");
-                    $rightside.append($language);*/
-
                     /*  Remember the user's last choice of theme using localStorage */
                     var themeKey = 'CRYPTPAD_CODE_THEME';
                     var lastTheme = localStorage.getItem(themeKey) || 'default';
 
-                    /*  Let the user select different themes */
-                    var $themeDropdown = $('<select>', {
-                        title: 'color theme',
-                        id: 'display-theme',
-                        'class': 'rightside-element'
-                    });
-                    Themes.forEach(function (o) {
-                        $themeDropdown.append($('<option>', {
-                            selected: o.name === lastTheme,
-                        }).val(o.name).text(o.name));
+                    var options = [];
+                    Themes.forEach(function (l) {
+                        options.push({
+                            tag: 'a',
+                            attributes: {
+                                'data-value': l.name,
+                                'href': '#',
+                            },
+                            content: l.name // Pretty name of the language value
+                        });
                     });
-
-
-                    $rightside.append($themeDropdown);
-
-                    var $theme = $bar.find('select#display-theme');
-
-                    setTheme(lastTheme, $theme);
-
-                    $theme.on('change', function () {
-                        var theme = $theme.val();
-                        console.log("Setting theme to %s", theme);
-                        setTheme(theme, $theme);
-                        // remember user choices
+                    var dropdownConfig = {
+                        text: 'Theme', // Button initial text
+                        options: options, // Entries displayed in the menu
+                        left: true, // Open to the left of the button
+                    };
+                    var $block = module.$theme = Cryptpad.createDropdown(dropdownConfig);
+                    var $button = $block.find('.buttonTitle');
+
+                    setTheme(lastTheme, $block);
+
+                    $block.find('a').click(function (e) {
+                        var theme = $(this).attr('data-value');
+                        setTheme(theme, $block);
+                        $button.text($(this).text());
                         localStorage.setItem(themeKey, theme);
                     });
+
+                    $rightside.append($block);
                 };
 
                 var configureColors = function () {

From 333800001892f790d8860800e518bba991034ca4 Mon Sep 17 00:00:00 2001
From: yflory <yann.flory@xwiki.com>
Date: Thu, 19 Jan 2017 12:49:40 +0100
Subject: [PATCH 2/4] Add Esc and Arrow icons in the slide's present mode

---
 customize.dist/main.css               | 23 +++++++++++++++++++++++
 customize.dist/src/less/cryptpad.less | 25 +++++++++++++++++++++++++
 www/slide/inner.html                  |  3 +++
 www/slide/slide.js                    | 25 +++++++++++++++++++++++++
 4 files changed, 76 insertions(+)

diff --git a/customize.dist/main.css b/customize.dist/main.css
index 0abf7e6bf..49af283c6 100644
--- a/customize.dist/main.css
+++ b/customize.dist/main.css
@@ -822,6 +822,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 {
diff --git a/customize.dist/src/less/cryptpad.less b/customize.dist/src/less/cryptpad.less
index 804dc6f98..7cf8f395a 100644
--- a/customize.dist/src/less/cryptpad.less
+++ b/customize.dist/src/less/cryptpad.less
@@ -478,6 +478,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; }
 
diff --git a/www/slide/inner.html b/www/slide/inner.html
index e3cdae259..1b8dbbf55 100644
--- a/www/slide/inner.html
+++ b/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>
diff --git a/www/slide/slide.js b/www/slide/slide.js
index cae81204f..501415cad 100644
--- a/www/slide/slide.js
+++ b/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) {

From 5d5d03163eaa7821b1f06802cb67d2d6927b60db Mon Sep 17 00:00:00 2001
From: yflory <yann.flory@xwiki.com>
Date: Thu, 19 Jan 2017 14:52:12 +0100
Subject: [PATCH 3/4] Add favicon and notifications in polls

---
 www/poll/index.html |   5 ++
 www/poll/main.js    | 177 +++++++++++++++++++++++++-------------------
 2 files changed, 107 insertions(+), 75 deletions(-)

diff --git a/www/poll/index.html b/www/poll/index.html
index 79e4c22b4..deac65f77 100644
--- a/www/poll/index.html
+++ b/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>
diff --git a/www/poll/main.js b/www/poll/main.js
index e97935dae..8ad1b4c0e 100644
--- a/www/poll/main.js
+++ b/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,87 +395,101 @@ define([
         });
     };
 
-            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) {
-                var users = userList ? userList.users : undefined;
-                //var userData = APP.proxy.info.userData;
-                for (var attrname in data) { userData[attrname] = data[attrname]; }
+    var unnotify = function () {
+        if (APP.tabNotification &&
+            typeof(APP.tabNotification.cancel) === 'function') {
+            APP.tabNotification.cancel();
+        }
+    };
 
-                if (users && users.length) {
-                    for (var userKey in userData) {
-                        if (users.indexOf(userKey) === -1) { delete userData[userKey]; }
-                    }
-                }
+    var notify = function () {
+        if (Visible.isSupported() && !Visible.currently()) {
+            unnotify();
+            APP.tabNotification = Notify.tab(1000, 10);
+        }
+    };
 
-                if(userList && typeof userList.onChange === "function") {
-                    userList.onChange(userData);
-                }
+    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) {
+        var users = userList ? userList.users : undefined;
+        //var userData = APP.proxy.info.userData;
+        for (var attrname in data) { userData[attrname] = data[attrname]; }
 
-                APP.proxy.info.userData = userData;
-            };
+        if (users && users.length) {
+            for (var userKey in userData) {
+                if (users.indexOf(userKey) === -1) { delete userData[userKey]; }
+            }
+        }
 
-            //var myData = {};
-            var getLastName = function (cb) {
-                Cryptpad.getAttribute('username', function (err, userName) {
-                    cb(err, userName || '');
-                });
-            };
+        if(userList && typeof userList.onChange === "function") {
+            userList.onChange(userData);
+        }
 
-            var setName = APP.setName = function (newName) {
-                if (typeof(newName) !== 'string') { return; }
-                var myUserNameTemp = Cryptpad.fixHTML(newName.trim());
-                if(myUserNameTemp.length > 32) {
-                    myUserNameTemp = myUserNameTemp.substr(0, 32);
-                }
-                var myUserName = myUserNameTemp;
-                var myID = APP.myID;
-                var myData = {};
-                myData[myID] = {
-                    name: myUserName
-                };
-                addToUserData(myData);
-                Cryptpad.setAttribute('username', newName, function (err, data) {
-                    if (err) {
-                        console.error("Couldn't set username");
-                        return;
-                    }
-                    APP.userName.lastName = myUserName;
-                    //change();
-                });
-            };
-
-            var updateTitle = function (newTitle) {
-                if (newTitle === document.title) { return; }
-                // Change the title now, and set it back to the old value if there is an error
-                var oldTitle = document.title;
-                document.title = newTitle;
-                Cryptpad.renamePad(newTitle, function (err, data) {
-                    if (err) {
-                        console.log("Couldn't set pad title");
-                        console.error(err);
-                        document.title = oldTitle;
-                        return;
-                    }
-                    document.title = data;
-                    APP.$bar.find('.' + Toolbar.constants.title).find('span.title').text(data);
-                    APP.$bar.find('.' + Toolbar.constants.title).find('input').val(data);
-                });
-            };
+        APP.proxy.info.userData = userData;
+    };
 
-            var updateDefaultTitle = function (defaultTitle) {
-                defaultName = defaultTitle;
-                APP.$bar.find('.' + Toolbar.constants.title).find('input').attr("placeholder", defaultName);
-            };
-            var renameCb = function (err, title) {
-                if (err) { return; }
-                document.title = title;
-                APP.proxy.info.title = title;
-            };
+    //var myData = {};
+    var getLastName = function (cb) {
+        Cryptpad.getAttribute('username', function (err, userName) {
+            cb(err, userName || '');
+        });
+    };
 
-            var suggestName = function (fallback) {
-                return document.title || defaultName || "";
-            };
+    var setName = APP.setName = function (newName) {
+        if (typeof(newName) !== 'string') { return; }
+        var myUserNameTemp = Cryptpad.fixHTML(newName.trim());
+        if(myUserNameTemp.length > 32) {
+            myUserNameTemp = myUserNameTemp.substr(0, 32);
+        }
+        var myUserName = myUserNameTemp;
+        var myID = APP.myID;
+        var myData = {};
+        myData[myID] = {
+            name: myUserName
+        };
+        addToUserData(myData);
+        Cryptpad.setAttribute('username', newName, function (err, data) {
+            if (err) {
+                console.error("Couldn't set username");
+                return;
+            }
+            APP.userName.lastName = myUserName;
+            //change();
+        });
+    };
+
+    var updateTitle = function (newTitle) {
+        if (newTitle === document.title) { return; }
+        // Change the title now, and set it back to the old value if there is an error
+        var oldTitle = document.title;
+        document.title = newTitle;
+        Cryptpad.renamePad(newTitle, function (err, data) {
+            if (err) {
+                console.log("Couldn't set pad title");
+                console.error(err);
+                document.title = oldTitle;
+                return;
+            }
+            document.title = data;
+            APP.$bar.find('.' + Toolbar.constants.title).find('span.title').text(data);
+            APP.$bar.find('.' + Toolbar.constants.title).find('input').val(data);
+        });
+    };
+
+    var updateDefaultTitle = function (defaultTitle) {
+        defaultName = defaultTitle;
+        APP.$bar.find('.' + Toolbar.constants.title).find('input').attr("placeholder", defaultName);
+    };
+    var renameCb = function (err, title) {
+        if (err) { return; }
+        document.title = title;
+        APP.proxy.info.title = title;
+    };
+
+    var suggestName = function (fallback) {
+        return document.title || defaultName || "";
+    };
 
 
     var copyObject = function (obj) {
@@ -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;
 

From 1e1bfd275cdb47719a2be9da970a98f71eab0f6c Mon Sep 17 00:00:00 2001
From: yflory <yann.flory@xwiki.com>
Date: Thu, 19 Jan 2017 15:17:21 +0100
Subject: [PATCH 4/4] Don't show the tab notification when someone has joined
 the pad

---
 www/code/main.js  |  5 +++--
 www/pad/main.js   | 14 +++++++++++++-
 www/slide/main.js |  4 +++-
 3 files changed, 19 insertions(+), 4 deletions(-)

diff --git a/www/code/main.js b/www/code/main.js
index 3af11316d..9a37efa99 100644
--- a/www/code/main.js
+++ b/www/code/main.js
@@ -684,8 +684,9 @@ define([
                         module.patchText(shjson2);
                     }
                 }
-
-                notify();
+                if (oldDoc !== remoteDoc) {
+                    notify();
+                }
             };
 
             var onAbort = config.onAbort = function (info) {
diff --git a/www/pad/main.js b/www/pad/main.js
index 852ad7cd5..60f64bce5 100644
--- a/www/pad/main.js
+++ b/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();
+
+                // 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) {
diff --git a/www/slide/main.js b/www/slide/main.js
index a44f11c85..b580c8ed8 100644
--- a/www/slide/main.js
+++ b/www/slide/main.js
@@ -768,7 +768,9 @@ define([
                 }
                 Slide.update(remoteDoc);
 
-                notify();
+                if (oldDoc !== newDoc) {
+                    notify();
+                }
             };
 
             var onAbort = config.onAbort = function (info) {