Merge branch 'onlyemoji-crash-fix' into 'develop'

Big emojis: fix possible web crash

See merge request soapbox-pub/soapbox-fe!602
error-boundary-browser-info
Alex Gleason 2021-07-10 17:46:46 +00:00
commit 284972aa76
1 zmienionych plików z 19 dodań i 12 usunięć

Wyświetl plik

@ -2,17 +2,24 @@
export const onlyEmoji = (node, limit = 1, ignoreMentions = true) => {
if (!node) return false;
// Remove mentions before checking content
if (ignoreMentions) {
node = node.cloneNode(true);
node.querySelectorAll('a.mention').forEach(m => m.parentNode.removeChild(m));
}
try {
// Remove mentions before checking content
if (ignoreMentions) {
node = node.cloneNode(true);
node.querySelectorAll('a.mention').forEach(m => m.parentNode.removeChild(m));
}
if (node.textContent.replaceAll(' ', '') !== '') return false;
const emojis = Array.from(node.querySelectorAll('img.emojione'));
if (emojis.length === 0) return false;
if (emojis.length > limit) return false;
const images = Array.from(node.querySelectorAll('img'));
if (images.length > emojis.length) return false;
return true;
if (node.textContent.replaceAll(' ', '') !== '') return false;
const emojis = Array.from(node.querySelectorAll('img.emojione'));
if (emojis.length === 0) return false;
if (emojis.length > limit) return false;
const images = Array.from(node.querySelectorAll('img'));
if (images.length > emojis.length) return false;
return true;
} catch (e) {
// Apparently some browsers can't handle `node.textContent.replaceAll`??
// If anything in here crashes, skipping it is inconsequential.
console.error(e);
return false;
}
};