merge: Don't convert empty HTML elements to MFM (resolves #982) (!1116)

View MR for information: https://activitypub.software/TransFem-org/Sharkey/-/merge_requests/1116

Closes #982

Approved-by: dakkar <dakkar@thenautilus.net>
Approved-by: Marie <github@yuugi.dev>
This commit is contained in:
Hazelnoot 2025-06-14 17:13:16 +00:00
commit 93885530c8

View file

@ -75,9 +75,8 @@ export class MfmService {
switch (node.tagName) { switch (node.tagName) {
case 'br': { case 'br': {
text += '\n'; text += '\n';
break; return;
} }
case 'a': { case 'a': {
const txt = getText(node); const txt = getText(node);
const rel = node.attribs.rel; const rel = node.attribs.rel;
@ -123,9 +122,16 @@ export class MfmService {
text += generateLink(); text += generateLink();
} }
break; return;
} }
}
// Don't produce invalid empty MFM
if (node.childNodes.length < 1) {
return;
}
switch (node.tagName) {
case 'h1': { case 'h1': {
text += '**【'; text += '**【';
appendChildren(node.childNodes); appendChildren(node.childNodes);