Subramanya Sastry has uploaded a new change for review.
https://gerrit.wikimedia.org/r/80513
Change subject: (Bug 53229): Dont strip sole child if it is an element
......................................................................
(Bug 53229): Dont strip sole child if it is an element
* en:Jay_Bouwmeester had an autoinserted <td> with an empty
<b></b> node inside and was being stripped by the dom cleanup
and added back in which got fostered out and tripped up RT-ing.
* The previous patch was missing a check for !DU.isElt(node.firstChild)
Added it in and that page now RTs cleanly.
Change-Id: Ic7b81bd99c95e31d251fbcf5fcea2320ea3082f0
---
M js/lib/dom.cleanup.js
1 file changed, 5 insertions(+), 4 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Parsoid
refs/changes/13/80513/1
diff --git a/js/lib/dom.cleanup.js b/js/lib/dom.cleanup.js
index 36354c6..f9a026d 100644
--- a/js/lib/dom.cleanup.js
+++ b/js/lib/dom.cleanup.js
@@ -35,10 +35,11 @@
if (dp) {
// Delete empty auto-inserted elements
var next = node.nextSibling;
- if ((node.childNodes.length === 0 ||
- node.childNodes.length === 1 &&
/^\s*$/.test(node.textContent)) &&
- dp.autoInsertedStart &&
dp.autoInsertedEnd) {
- if (node.childNodes.length === 1) {
+ if (dp.autoInsertedStart && dp.autoInsertedEnd &&
+ (node.childNodes.length === 0 ||
+ node.childNodes.length === 1 &&
!DU.isElt(node.firstChild) && /^\s*$/.test(node.textContent)))
+ {
+ if (node.firstChild) {
// migrate the ws out
node.parentNode.insertBefore(node.firstChild, node);
}
--
To view, visit https://gerrit.wikimedia.org/r/80513
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic7b81bd99c95e31d251fbcf5fcea2320ea3082f0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Parsoid
Gerrit-Branch: master
Gerrit-Owner: Subramanya Sastry <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
