diff --git a/customize.dist/src/less2/include/sidebar-layout.less b/customize.dist/src/less2/include/sidebar-layout.less index 1b0c75932..564067fb7 100644 --- a/customize.dist/src/less2/include/sidebar-layout.less +++ b/customize.dist/src/less2/include/sidebar-layout.less @@ -69,7 +69,6 @@ background: @cp_sidebar-right-bg; color: @cp_sidebar-right-fg; overflow: auto; - //padding-bottom: 200px; // XXX what was the intent behind this? // Following rules are only in settings .cp-sidebarlayout-element { diff --git a/customize.dist/src/less2/include/toolbar.less b/customize.dist/src/less2/include/toolbar.less index 2092d72cc..6ccec507d 100644 --- a/customize.dist/src/less2/include/toolbar.less +++ b/customize.dist/src/less2/include/toolbar.less @@ -532,7 +532,13 @@ &> button { height: @toolbar_line-height; width: @toolbar_line-height; - span { font-size: unset; } + span { + .avatar_vars(36px); + font-size: @avatar-font-size; + .animal { + font-size: @avatar-font-size-animal; + } + } } &> button.cp-avatar.cp-avatar { media-tag { @@ -862,7 +868,6 @@ align-items: center; .animal { font-size: @avatar-font-size-animal; - } } &.cp-avatar { diff --git a/www/common/boot2.js b/www/common/boot2.js index ada5b794c..d143da87d 100644 --- a/www/common/boot2.js +++ b/www/common/boot2.js @@ -34,6 +34,7 @@ try { define([ '/common/requireconfig.js' ], function (RequireConfig) { + require.config(RequireConfig()); // most of CryptPad breaks if you don't support isArray @@ -91,4 +92,10 @@ define([ } catch (e) { console.error(e); failStore(); } require([document.querySelector('script[data-bootload]').getAttribute('data-bootload')]); + if (typeof(Promise) !== 'function') { + setTimeout(function () { + var s = "Internet Explorer is not supported anymore, including by Microsoft.\n\nMost of CryptPad's collaborative functionality requires a modern browser to work.\n\nWe recommend Mozilla Firefox."; + window.alert(s); + }); + } }); diff --git a/www/common/common-messaging.js b/www/common/common-messaging.js index 1e6e26ce3..4a3eb2a91 100644 --- a/www/common/common-messaging.js +++ b/www/common/common-messaging.js @@ -18,7 +18,7 @@ define([ curvePublic: proxy.curvePublic, notifications: Util.find(proxy, ['mailboxes', 'notifications', 'channel']), avatar: proxy.profile && proxy.profile.avatar, - uid: proxy.uid, // XXX test without this and see if it breaks things + uid: proxy.uid, }; if (hash === false) { delete data.channel; } return data; diff --git a/www/common/inner/common-mediatag.js b/www/common/inner/common-mediatag.js index 21cb25e82..33e5d74f7 100644 --- a/www/common/inner/common-mediatag.js +++ b/www/common/inner/common-mediatag.js @@ -90,17 +90,12 @@ define([ // https://emojipedia.org/nature/ var ANIMALS = AppConfig.emojiAvatars || []; - var getRandomAnimal = function () { // XXX should never actually happen? - if (!ANIMALS.length) { return ''; } - return ANIMALS[Math.floor(Math.random() * ANIMALS.length)]; - }; - var getPseudorandomAnimal = MT.getPseudorandomAnimal = function (seed) { if (!ANIMALS.length) { return ''; } - if (typeof(seed) !== 'string') { return getRandomAnimal(); } - seed = seed.replace(/\D/g, '').slice(0, 10); // XXX possible optimization for on-wire uid + if (typeof(seed) !== 'string') { return; } + seed = seed.replace(/\D/g, '').slice(0, 10); // TODO possible optimization for on-wire uid seed = parseInt(seed); - if (!seed) { return getRandomAnimal(); } + if (!seed) { return; } return ANIMALS[seed % ANIMALS.length] || ''; }; @@ -141,7 +136,7 @@ define([ var $avatar = $('', { 'class': 'cp-avatar-default' + (animal_avatar? ' animal': ''), - // XXX prevents screenreaders from trying to describe this + // this prevents screenreaders from trying to describe this alt: '', 'aria-hidden': true, }).text(text); @@ -196,7 +191,7 @@ define([ var $img = $(mt).appendTo($container); MT.displayMediatagImage(common, $img, function (err, $image) { if (err) { return void console.error(err); } - centerImage($img, $image); // XXX add alt="" (unless the media-tag has an alt attr) + centerImage($img, $image); }); }); } diff --git a/www/common/sframe-common.js b/www/common/sframe-common.js index 3575c95b0..b7efd9f4f 100644 --- a/www/common/sframe-common.js +++ b/www/common/sframe-common.js @@ -235,9 +235,6 @@ define([ }; }; - funcs.getAuthorId = function () { // XXX - }; - var authorUid = function(existing) { if (!Array.isArray(existing)) { existing = []; } var n; @@ -263,7 +260,7 @@ define([ }); return uid || authorUid(existing); } - // XXX this should check for a matching curvePublic / uid if: + // TODO this should check for a matching curvePublic / uid if: // 1. you are logged in OR // 2. you have a token // so that users that register recognize comments from before diff --git a/www/common/toolbar.js b/www/common/toolbar.js index 5b359a323..5f1c2ece5 100644 --- a/www/common/toolbar.js +++ b/www/common/toolbar.js @@ -164,6 +164,7 @@ MessengerUI, Messages, Pages) { }); }; var showColors = false; + Messages.userlist_visitProfile = "Visit {0}'s profile"; // XXX "'s" is incorrect for names that end in "s" in English... don't care? var updateUserList = function (toolbar, config, forceOffline) { if (!config.displayed || config.displayed.indexOf('userlist') === -1) { return; } if (toolbar.isAlone) { return; } @@ -249,6 +250,7 @@ MessengerUI, Messages, Pages) { var friendRequests = Common.getFriendRequests(); // Friend requests received editUsersNames.forEach(function (data) { var name = data.name || Messages.anonymous; + var safeName = Util.fixHTML(name); var $span = $('', {'class': 'cp-avatar'}); if (data.color && showColors) { $span.css('border-color', data.color); @@ -323,7 +325,7 @@ MessengerUI, Messages, Pages) { $('