| DannyS712 | |
| Jul 10 2020, 7:36 AM |
| F32270220: image.png | |
| Sep 11 2020, 1:49 PM |
| F32270014: image.png | |
| Sep 11 2020, 12:29 PM |
Background:
On enwiki, SineBot adds missing signatures for ips and new users. The bot substitutes a template that notes the comment was unsigned, and includes the editor and the time of the comment. After the template, the bot notes <!--Autosigned by SineBot-->
User story:
As someone using the reply tool,
When I reply to a post that was autosigned by SineBot,
My reply should go after the <!--Autosigned by SineBot--> comment
Example: http://en.wikipedia.org/wiki/Special:Diff/966900777
Change 615551 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/DiscussionTools@master] Better handle HTML comments following replies
Change 615551 merged by jenkins-bot:
[mediawiki/extensions/DiscussionTools@master] Better handle HTML comments following replies
In our Planning meeting the editing team decided to reopen this task because the issue reoccurred
I can reproduce this iff the comment being replied to is unindented and the signature is wrapped in a <small> tag:
The issues appears to be whitespace between the end of the paragraph and the comment. Parsoid appears to put the whitespace in an inconsistent place. In the list case Parsoid appears to be losing the whitespace completely, which was helping us but is a bug:
signature1 <!--comment--> <small>signature2</small> <!--comment--> : signature3 <!--comment--> : <small>signature4</small> <!--comment-->
<p>signature1 </p><!--comment--> <p><small>signature2</small></p> <!--comment--> <dl><dd>signature3<!--comment--></dd> <dd><small>signature4</small><!--comment--></dd></dl>
Case signature2 has whitespace after the </p> which is triggering the bug
CC @ssastry, @cscott who may want to look into the dropped whitespace and inconsistencies.
Change 626689 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/DiscussionTools@master] Skip over whitespace when looking for trailing comments
"Death by a thousand whitespace cuts" is what we should name the tracking bug.
It is not. See T157418: RFC: Make some aspects of Tidy's whitespace stripping behavior part of wikitext parsing "spec". The core parser strips comments from source which leaves the list item with whitespace which Tidy would then strip out. So, Parsoid effectively mimics that behavior as per T157418.
wikitextsignature1 <!--comment--> <small>signature2</small> <!--comment-->HTML<p>signature1 </p><!--comment--> <p><small>signature2</small></p> <!--comment-->Case signature2 has whitespace after the </p> which is triggering the bug
Regarding the inconsistency, the paragraph wrapping code is a mess because of T134469: doBlockLevels() inserts <p> and </p> randomly with no regard for HTML validity and the need to deal with whitespace, comments, templates, rendering-transparent metadata like categories, blah. But, I suspect the "state machine" logic does whatever is expedient wrt whitespace & comments wrt to where to insert the </p> tag in the buffered token stream. We can tweak it if necessary. But, if nothing is broken on your end because of it, I would much rather not worry about it on our end.
Change 626689 merged by jenkins-bot:
[mediawiki/extensions/DiscussionTools@master] Skip over whitespace when looking for trailing comments
I'm assuming nothing is broken on our end considering this now seems to be functioning as we expect [i] – would this be accurate for me to think, @Esanders ?
