In these edits, the replies were added inside wrapper <div> node for "The Signpost" delivery, instead of after it:
I'm pretty sure that this worked correctly after T250126, it might be a regression.
matmarex | |
Dec 4 2021, 1:29 AM |
F34951333: Screenshot 2022-02-14 at 14.55.30.png | |
Feb 14 2022, 1:57 PM |
F34951328: Screenshot 2022-02-14 at 14.52.14.png | |
Feb 14 2022, 1:57 PM |
F34951331: Screenshot 2022-02-14 at 14.53.58.png | |
Feb 14 2022, 1:57 PM |
F34926095: image.png | |
Jan 21 2022, 8:33 PM |
F34926093: image.png | |
Jan 21 2022, 8:33 PM |
F34926097: image.png | |
Jan 21 2022, 8:33 PM |
F34825892: image.png | |
Dec 4 2021, 1:31 AM |
F34825895: image.png | |
Dec 4 2021, 1:31 AM |
In these edits, the replies were added inside wrapper <div> node for "The Signpost" delivery, instead of after it:
I'm pretty sure that this worked correctly after T250126, it might be a regression.
The issue is also apparent when using the reply tool before saving (note how the font from the message is applied to our interface):
And in a few other messages on https://en.wikipedia.org/wiki/User_talk:Dylsss:
CC @Dylsss, in case you find this interesting, since you were testing this in that edit I found, and since you're active on Phabricator. I found the edits using https://dtcheck.toolforge.org/, which is a tool we built for tracking down dirty diffs in edits made with the reply tool.
Change 743525 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):
[mediawiki/extensions/DiscussionTools@master] [WIP] Ignore some invisible nodes when looking for comment frames
Change 747812 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):
[mediawiki/extensions/DiscussionTools@master] Test cases for adding replies outside of frames
Change 747812 merged by jenkins-bot:
[mediawiki/extensions/DiscussionTools@master] Test cases for adding replies outside of frames
Change 743525 merged by jenkins-bot:
[mediawiki/extensions/DiscussionTools@master] Ignore some invisible nodes when looking for comment frames
@Ryasmeen noticed that it's only partially fixed:
Before | After | Status |
---|---|---|
fixed | ||
NOT FIXED | ||
fixed |
Change 756637 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):
[mediawiki/extensions/DiscussionTools@master] One more tweak for comparing comment ranges to transclusion/DOM ranges
Change 756637 merged by jenkins-bot:
[mediawiki/extensions/DiscussionTools@master] One more tweak for comparing comment ranges to transclusion/DOM ranges
All good now. Following the same order as above, the fixed versions look like:
@matmarex , let me know if this doesn't align with the expected result.