jenkins-bot has submitted this change and it was merged.
Change subject: Followup to 51a6fb8c: Handle tags with zero-wikitext-tag-width
......................................................................
Followup to 51a6fb8c: Handle tags with zero-wikitext-tag-width
* Eliminates RT regression from it:Seconda_battaglia_dell%27Artois
where a stray </p> was serialized back.
Not sure why the stray </p> was generated in the HTML in the
first place, but it shouldn't be auto-marked 'stx:html'
Change-Id: I7f64757df150482805a52b6a9c3f840f26b927a6
---
M js/lib/dom.markTreeBuilderFixups.js
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
GWicke: Looks good to me, approved
jenkins-bot: Verified
diff --git a/js/lib/dom.markTreeBuilderFixups.js
b/js/lib/dom.markTreeBuilderFixups.js
index 22674b7..aa71748 100644
--- a/js/lib/dom.markTreeBuilderFixups.js
+++ b/js/lib/dom.markTreeBuilderFixups.js
@@ -183,8 +183,9 @@
// If the start was
auto-inserted and it does not have a
// closing-tag in wikitext,
clearly the end-tag came from
// a literal HTML end tag (ex:
</tr>) from the source.
+ // But, ignore HTML tags that
dont have wikitext tags at all.
var wtTagWidth =
Consts.WT_TagWidths[cNodeName];
- if (wtTagWidth && wtTagWidth[1]
=== 0) {
+ if (wtTagWidth && wtTagWidth[0]
!== 0 && wtTagWidth[1] === 0) {
dp.stx = 'html';
}
}
--
To view, visit https://gerrit.wikimedia.org/r/81663
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7f64757df150482805a52b6a9c3f840f26b927a6
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Parsoid
Gerrit-Branch: master
Gerrit-Owner: Subramanya Sastry <[email protected]>
Gerrit-Reviewer: GWicke <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
