Foundry376/Mailspring
1
1
Fork 0
mirror of https://github.com/Foundry376/Mailspring.git synced 2025-10-02 17:34:49 +08:00
Code Issues Projects Releases Packages Wiki Activity

Don’t remove top level <br> tags that separate text nodes (Apple calendar alerts)

Browse source
This commit is contained in:
Ben Gotow 2020-05-24 18:59:15 -05:00
parent 4c6ab52e5a
commit df26391c57
1 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
app/src/services/quoted-html-transformer.ts
View file

@ -105,12 +105,13 @@ class QuotedHTMLTransformer {
return; return;
} }
// Find back-to-back <br><br> at the top level and de-duplicate them // Find back-to-back <br><br> at the top level and de-duplicate them. Note that
const { children } = doc.body; // some emails contain TEXT<br>TEXT<br>TEXT, so the only ELEMENT children may be the <brs>
const nodes = doc.body.childNodes;
const extraTailBrTags = []; const extraTailBrTags = [];
for (let i = children.length - 1; i >= 0; i--) { for (let i = nodes.length - 1; i >= 0; i--) {
const curr = children[i]; const curr = nodes[i];
const next = children[i - 1]; const next = nodes[i - 1];
if (curr && curr.nodeName === 'BR' && next && next.nodeName === 'BR') { if (curr && curr.nodeName === 'BR' && next && next.nodeName === 'BR') {
extraTailBrTags.push(curr); extraTailBrTags.push(curr);
} else { } else {