Trevor Parscal has submitted this change and it was merged.

Change subject: (bug 37980) Cutting all text and repasting it breaks the editor
......................................................................


(bug 37980) Cutting all text and repasting it breaks the editor

This was caused by a bug in fixupInsertion that caused it to believe
that inserting something like "a</p><p>b</p><p>c" into the middle of an
empty paragraph was invalid.

This commit fixes the fixupInsertion bug, which fixes the
select-all-cut-paste behavior in Chrome. It's still broken in Firefox
because of selection-related issues, but I'll split that out into a
different bug report.

Change-Id: I767f5d37ec7e511778ae9ca8283ec4b26c728298
---
M modules/ve/dm/ve.dm.Document.js
M tests/ve/dm/ve.dm.Transaction.test.js
2 files changed, 71 insertions(+), 44 deletions(-)

Approvals:
  Trevor Parscal: Verified; Looks good to me, approved


--
To view, visit https://gerrit.wikimedia.org/r/14398
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I767f5d37ec7e511778ae9ca8283ec4b26c728298
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Catrope <[email protected]>
Gerrit-Reviewer: Catrope <[email protected]>
Gerrit-Reviewer: Trevor Parscal <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to