nettika/geary
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Check class name in ConversationPageState.isDescendantOf

Browse source
This commit is contained in:
Alex Henrie 2019-01-08 20:15:48 -07:00
parent 3000334955
commit b9f984e65e
1 changed files with 15 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

21
ui/conversation-web-view.js
View file

@ -82,7 +82,7 @@ ConversationPageState.prototype = {
// Only insert into a quote container if the element is a // Only insert into a quote container if the element is a
// top level blockquote // top level blockquote
if (!ConversationPageState.isDescendantOf(blockquote, "BLOCKQUOTE")) { if (!ConversationPageState.isDescendantOf(blockquote, "blockquote")) {
let quoteHeight = blockquote.offsetHeight; let quoteHeight = blockquote.offsetHeight;
// Only make the quote it controllable if it is tall enough // Only make the quote it controllable if it is tall enough
@ -171,7 +171,7 @@ ConversationPageState.prototype = {
let div = possibleSigs.item(i); let div = possibleSigs.item(i);
let innerHTML = div.innerHTML; let innerHTML = div.innerHTML;
if ((sigRegex.test(innerHTML) || alternateSigRegex.test(innerHTML)) && if ((sigRegex.test(innerHTML) || alternateSigRegex.test(innerHTML)) &&
!ConversationPageState.isDescendantOf(div, "BLOCKQUOTE")) { !ConversationPageState.isDescendantOf(div, "blockquote")) {
break; break;
} }
} }
@ -206,7 +206,7 @@ ConversationPageState.prototype = {
// so that new lines are preserved. // so that new lines are preserved.
let dummy = document.createElement("DIV"); let dummy = document.createElement("DIV");
let includeDummy = false; let includeDummy = false;
if (ConversationPageState.isDescendantOf(ancestor, ".plaintext")) { if (ConversationPageState.isDescendantOf(ancestor, "div", "plaintext")) {
dummy.classList.add("plaintext"); dummy.classList.add("plaintext");
dummy.setAttribute("style", "white-space: pre-wrap;"); dummy.setAttribute("style", "white-space: pre-wrap;");
includeDummy = true; includeDummy = true;
@ -321,11 +321,20 @@ ConversationPageState.isDeceptiveText = function(text, href) {
return ConversationPageState.NOT_DECEPTIVE; return ConversationPageState.NOT_DECEPTIVE;
}; };
ConversationPageState.isDescendantOf = function(node, ancestorTag) { /**
* See if this element has an ancestor with the given tag and class.
*
* ancestorTag must be all lowercase.
*
* If ancestorClass is null, no class checking is done.
*/
ConversationPageState.isDescendantOf = function(node, ancestorTag, ancestorClass = null) {
let ancestor = node.parentNode; let ancestor = node.parentNode;
while (ancestor != null) { while (ancestor != null) {
if (ancestor.tagName == ancestorTag) { if (ancestor.tagName.toLowerCase() == ancestorTag) {
return true; if (!ancestorClass || ancestor.classList.contains(ancestorClass)) {
return true;
}
} }
ancestor = ancestor.parentNode; ancestor = ancestor.parentNode;
} }