mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/DiscussionTools
synced 2024-11-24 08:23:52 +00:00
e5e6fdd3af
Native Range objects are automatically updated when the DOM elements they refer to are affected (e.g. detached from the DOM, or their offset changes because of siblings being added/removed). This seemed harmless or maybe even slightly useful, but it turns out it conflicts with VisualEditor, which has to wrap the entire page in a new DOM node when it opens (and unwrap it when it closes), effectively temporarily detaching it from the DOM, which destroys all our ranges. Just use a plain object that stores the same data as a Range. And when we need to use Range's API, we can simply construct a temporary one. Bug: T241861 Change-Id: Iee64aa3d667877265ef8a59293c202e6478d7fb6 |
||
---|---|---|
.. | ||
lib | ||
.eslintrc.json | ||
CommentTarget.js | ||
CommentTargetWidget.js | ||
CommentTargetWidget.less | ||
controller.js | ||
dt.debug.js | ||
dt.init.js | ||
dt.init.less | ||
dt.ui.ReplyWidget.js | ||
dt.ui.ReplyWidget.less | ||
dt.ui.ReplyWidgetPlain.js | ||
dt.ui.ReplyWidgetVisual.js | ||
highlighter.css | ||
highlighter.js | ||
modifier.js | ||
parser.js |