| Esanders | |
| Mar 9 2022, 2:43 PM |
| F34993852: image.png | |
| Mar 9 2022, 9:51 PM |
Visit https://en.wikipedia.beta.wmflabs.org/w/index.php?title=User_talk:ESanders_(WMF)/Sandbox&oldid=541311 while logged and and hit [reply],
Observe a new <dl> is attached to the DOM as a sibling of the main mw-parser-output node.
This doesn't happen if you are logged in, or viewing the latest revision of the page (?!)
If you use the edit-conflict-reload feature, the "Loading..." message does not get cleared as we only update the mw-parser-output node.
| Project | Branch | Lines +/- | Subject | |
|---|---|---|---|---|
| mediawiki/extensions/DiscussionTools | master | +97 -12 | Don't allow the root node to be treated like a comment frame |
Change 769536 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):
[mediawiki/extensions/DiscussionTools@master] Don't allow the root node to be treated like a comment frame
This is caused by the code that lets the reply tool (and the real reply) appear outside of a frame like in this example: https://en.wikipedia.beta.wmflabs.org/w/index.php?title=Talk:T303396&oldid=541380
To achieve this, CommentModifier tries to find a set of sibling nodes that almost exactly matches the comment's range. There is some ugly code implementing the "almost exactly" part, basically it ignores various nodes according to various made up rules – for example, our own [reply] buttons and markers, and HTML comments (check the wikitext source for an example).
There were two bugs in that code:
Change 769536 merged by jenkins-bot:
[mediawiki/extensions/DiscussionTools@master] Don't allow the root node to be treated like a comment frame
The number of <dl> elements remain the same after hitting/toggling [reply]. See https://photos.app.goo.gl/9Sp6ofL6bQaoYTHS7
Looks good to me.