wikimedia/mediawiki-extensions-DiscussionTools
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/DiscussionTools synced 2024-11-16 20:58:28 +00:00
Code Issues Packages Projects Releases Wiki Activity
mediawiki-extensions-Discus.../tests/cases/wrappers/wrappers-formattedreply.html

65 lines
28 KiB
HTML
Raw Normal View History

Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<h2 id="paragraph" data-mw-comment="{&quot;type&quot;:&quot;heading&quot;,&quot;level&quot;:0,&quot;id&quot;:&quot;h-paragraph-2020-01-22T23:19:00.000Z&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-paragraph&quot;],&quot;headingLevel&quot;:2,&quot;placeholderHeading&quot;:false}" class="ext-discussiontools-init-section"><span data-mw-comment-start="" id="h-paragraph-2020-01-22T23:19:00.000Z"></span>paragraph<span data-mw-comment-end="h-paragraph-2020-01-22T23:19:00.000Z"></span><span class="ext-discussiontools-init-section-subscribe mw-editsection-like"><span class="ext-discussiontools-init-section-subscribe-bracket">[</span><a href="" class="ext-discussiontools-init-section-subscribe-link" role="button" tabindex="0" data-mw-comment-name="h-Matma_Rex-2020-01-22T23:19:00.000Z" title="(discussiontools-topicsubscription-button-subscribe-tooltip)">(discussiontools-topicsubscription-button-subscribe)</a><span class="ext-discussiontools-init-section-subscribe-bracket">]</span></span></h2>
<p><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-paragraph"></span>blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-paragraph&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-paragraph"></span></p>
Fix detecting decorative comment frames with whitespace As a result of 0fc71f60cd464e9cf54c4048985971953503ce51, "empty" text nodes (containing only whitespace) at the end of the comment may be inside the comment's range, and trying to ignore them caused the ranges not to match and the frame not to be detected. Now the code works whether they're inside the comment's range or not. Add a test case for wrapped discussion comments with HTML comments and with whitespace. Bug: T250126 Bug: T268407 Change-Id: I2217ff5a635fd1c9c9e803f46795b1bfb3d17535
2020-12-08 22:46:11 +00:00
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<h2 id="preformatted" data-mw-comment="{&quot;type&quot;:&quot;heading&quot;,&quot;level&quot;:0,&quot;id&quot;:&quot;h-preformatted-2020-01-22T23:19:00.000Z&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-preformatted&quot;],&quot;headingLevel&quot;:2,&quot;placeholderHeading&quot;:false}" class="ext-discussiontools-init-section"><span data-mw-comment-start="" id="h-preformatted-2020-01-22T23:19:00.000Z"></span>preformatted<span data-mw-comment-end="h-preformatted-2020-01-22T23:19:00.000Z"></span><span class="ext-discussiontools-init-section-subscribe mw-editsection-like"><span class="ext-discussiontools-init-section-subscribe-bracket">[</span><a href="" class="ext-discussiontools-init-section-subscribe-link" role="button" tabindex="0" data-mw-comment-name="h-Matma_Rex-2020-01-22T23:19:00.000Z" title="(discussiontools-topicsubscription-button-subscribe-tooltip)">(discussiontools-topicsubscription-button-subscribe)</a><span class="ext-discussiontools-init-section-subscribe-bracket">]</span></span></h2>
<pre><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-preformatted"></span>blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-preformatted&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-preformatted"></span></pre>
Fix detecting decorative comment frames with whitespace As a result of 0fc71f60cd464e9cf54c4048985971953503ce51, "empty" text nodes (containing only whitespace) at the end of the comment may be inside the comment's range, and trying to ignore them caused the ranges not to match and the frame not to be detected. Now the code works whether they're inside the comment's range or not. Add a test case for wrapped discussion comments with HTML comments and with whitespace. Bug: T250126 Bug: T268407 Change-Id: I2217ff5a635fd1c9c9e803f46795b1bfb3d17535
2020-12-08 22:46:11 +00:00
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<h2 id="div_with_one_comment" data-mw-comment="{&quot;type&quot;:&quot;heading&quot;,&quot;level&quot;:0,&quot;id&quot;:&quot;h-div_with_one_comment-2020-01-22T23:19:00.000Z&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment&quot;],&quot;headingLevel&quot;:2,&quot;placeholderHeading&quot;:false}" class="ext-discussiontools-init-section"><span data-mw-comment-start="" id="h-div_with_one_comment-2020-01-22T23:19:00.000Z"></span>div with one comment<span data-mw-comment-end="h-div_with_one_comment-2020-01-22T23:19:00.000Z"></span><span class="ext-discussiontools-init-section-subscribe mw-editsection-like"><span class="ext-discussiontools-init-section-subscribe-bracket">[</span><a href="" class="ext-discussiontools-init-section-subscribe-link" role="button" tabindex="0" data-mw-comment-name="h-Matma_Rex-2020-01-22T23:19:00.000Z" title="(discussiontools-topicsubscription-button-subscribe-tooltip)">(discussiontools-topicsubscription-button-subscribe)</a><span class="ext-discussiontools-init-section-subscribe-bracket">]</span></span></h2>
Give comments IDs so they can be scrolled to with hash links Bug: T265268 Change-Id: Idb985ed38bdb74e23cb7840899a61dc919f05f6f
2021-03-10 17:25:52 +00:00
<div style="background: beige;"><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment"></span>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment"></span>
Fix detecting decorative comment frames with whitespace As a result of 0fc71f60cd464e9cf54c4048985971953503ce51, "empty" text nodes (containing only whitespace) at the end of the comment may be inside the comment's range, and trying to ignore them caused the ranges not to match and the frame not to be detected. Now the code works whether they're inside the comment's range or not. Add a test case for wrapped discussion comments with HTML comments and with whitespace. Bug: T250126 Bug: T268407 Change-Id: I2217ff5a635fd1c9c9e803f46795b1bfb3d17535
2020-12-08 22:46:11 +00:00
</div>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<h2 id="two_divs_with_one_comment" data-mw-comment="{&quot;type&quot;:&quot;heading&quot;,&quot;level&quot;:0,&quot;id&quot;:&quot;h-two_divs_with_one_comment-2020-01-22T23:19:00.000Z&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-two_divs_with_one_comment&quot;],&quot;headingLevel&quot;:2,&quot;placeholderHeading&quot;:false}" class="ext-discussiontools-init-section"><span data-mw-comment-start="" id="h-two_divs_with_one_comment-2020-01-22T23:19:00.000Z"></span>two divs with one comment<span data-mw-comment-end="h-two_divs_with_one_comment-2020-01-22T23:19:00.000Z"></span><span class="ext-discussiontools-init-section-subscribe mw-editsection-like"><span class="ext-discussiontools-init-section-subscribe-bracket">[</span><a href="" class="ext-discussiontools-init-section-subscribe-link" role="button" tabindex="0" data-mw-comment-name="h-Matma_Rex-2020-01-22T23:19:00.000Z" title="(discussiontools-topicsubscription-button-subscribe-tooltip)">(discussiontools-topicsubscription-button-subscribe)</a><span class="ext-discussiontools-init-section-subscribe-bracket">]</span></span></h2>
Give comments IDs so they can be scrolled to with hash links Bug: T265268 Change-Id: Idb985ed38bdb74e23cb7840899a61dc919f05f6f
2021-03-10 17:25:52 +00:00
<div style="background: beige;"><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-two_divs_with_one_comment"></span>
More test cases for comment wrappers (multiple siblings) For a moment I doubted if we handle this case correctly, but in fact I didn't botch that code *this* badly. Change-Id: I5a9d142e4bd97ac40aa388bb43b65ab1286e3f18
2021-02-02 23:03:45 +00:00
blah
</div>
<div style="background: beige;">
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-two_divs_with_one_comment&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-two_divs_with_one_comment"></span>
More test cases for comment wrappers (multiple siblings) For a moment I doubted if we handle this case correctly, but in fact I didn't botch that code *this* badly. Change-Id: I5a9d142e4bd97ac40aa388bb43b65ab1286e3f18
2021-02-02 23:03:45 +00:00
</div>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<h2 id="div_with_one_comment-htmlcomments" data-mw-comment="{&quot;type&quot;:&quot;heading&quot;,&quot;level&quot;:0,&quot;id&quot;:&quot;h-div_with_one_comment-htmlcomments-2020-01-22T23:19:00.000Z&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment-htmlcomments&quot;],&quot;headingLevel&quot;:2,&quot;placeholderHeading&quot;:false}" class="ext-discussiontools-init-section"><span data-mw-comment-start="" id="h-div_with_one_comment-htmlcomments-2020-01-22T23:19:00.000Z"></span>div with one comment, and some HTML comments<span data-mw-comment-end="h-div_with_one_comment-htmlcomments-2020-01-22T23:19:00.000Z"></span><span class="ext-discussiontools-init-section-subscribe mw-editsection-like"><span class="ext-discussiontools-init-section-subscribe-bracket">[</span><a href="" class="ext-discussiontools-init-section-subscribe-link" role="button" tabindex="0" data-mw-comment-name="h-Matma_Rex-2020-01-22T23:19:00.000Z" title="(discussiontools-topicsubscription-button-subscribe-tooltip)">(discussiontools-topicsubscription-button-subscribe)</a><span class="ext-discussiontools-init-section-subscribe-bracket">]</span></span></h2>
Fix detecting decorative comment frames with whitespace As a result of 0fc71f60cd464e9cf54c4048985971953503ce51, "empty" text nodes (containing only whitespace) at the end of the comment may be inside the comment's range, and trying to ignore them caused the ranges not to match and the frame not to be detected. Now the code works whether they're inside the comment's range or not. Add a test case for wrapped discussion comments with HTML comments and with whitespace. Bug: T250126 Bug: T268407 Change-Id: I2217ff5a635fd1c9c9e803f46795b1bfb3d17535
2020-12-08 22:46:11 +00:00
<div style="background: beige;">
Give comments IDs so they can be scrolled to with hash links Bug: T265268 Change-Id: Idb985ed38bdb74e23cb7840899a61dc919f05f6f
2021-03-10 17:25:52 +00:00
<i><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment-htmlcomments"></span>hello</i><br/>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<small>blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)</small><!-- blah --><span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment-htmlcomments&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment-htmlcomments"></span>
Fix detecting decorative comment frames with whitespace As a result of 0fc71f60cd464e9cf54c4048985971953503ce51, "empty" text nodes (containing only whitespace) at the end of the comment may be inside the comment's range, and trying to ignore them caused the ranges not to match and the frame not to be detected. Now the code works whether they're inside the comment's range or not. Add a test case for wrapped discussion comments with HTML comments and with whitespace. Bug: T250126 Bug: T268407 Change-Id: I2217ff5a635fd1c9c9e803f46795b1bfb3d17535
2020-12-08 22:46:11 +00:00
</div><!-- blah -->
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<h2 id="two_divs_with_one_comment-htmlcomments" data-mw-comment="{&quot;type&quot;:&quot;heading&quot;,&quot;level&quot;:0,&quot;id&quot;:&quot;h-two_divs_with_one_comment-htmlcomments-2020-01-22T23:19:00.000Z&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-two_divs_with_one_comment-htmlcomments&quot;],&quot;headingLevel&quot;:2,&quot;placeholderHeading&quot;:false}" class="ext-discussiontools-init-section"><span data-mw-comment-start="" id="h-two_divs_with_one_comment-htmlcomments-2020-01-22T23:19:00.000Z"></span>two divs with one comment, and some HTML comments<span data-mw-comment-end="h-two_divs_with_one_comment-htmlcomments-2020-01-22T23:19:00.000Z"></span><span class="ext-discussiontools-init-section-subscribe mw-editsection-like"><span class="ext-discussiontools-init-section-subscribe-bracket">[</span><a href="" class="ext-discussiontools-init-section-subscribe-link" role="button" tabindex="0" data-mw-comment-name="h-Matma_Rex-2020-01-22T23:19:00.000Z" title="(discussiontools-topicsubscription-button-subscribe-tooltip)">(discussiontools-topicsubscription-button-subscribe)</a><span class="ext-discussiontools-init-section-subscribe-bracket">]</span></span></h2>
Give comments IDs so they can be scrolled to with hash links Bug: T265268 Change-Id: Idb985ed38bdb74e23cb7840899a61dc919f05f6f
2021-03-10 17:25:52 +00:00
<div style="background: beige;"><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-two_divs_with_one_comment-htmlcomments"></span>
More test cases for comment wrappers (multiple siblings) For a moment I doubted if we handle this case correctly, but in fact I didn't botch that code *this* badly. Change-Id: I5a9d142e4bd97ac40aa388bb43b65ab1286e3f18
2021-02-02 23:03:45 +00:00
blah
</div>
<!-- blah -->
<div style="background: beige;">
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-two_divs_with_one_comment-htmlcomments&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-two_divs_with_one_comment-htmlcomments"></span>
More test cases for comment wrappers (multiple siblings) For a moment I doubted if we handle this case correctly, but in fact I didn't botch that code *this* badly. Change-Id: I5a9d142e4bd97ac40aa388bb43b65ab1286e3f18
2021-02-02 23:03:45 +00:00
</div><!-- blah -->
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<h2 id="div_with_one_comment-list_end" data-mw-comment="{&quot;type&quot;:&quot;heading&quot;,&quot;level&quot;:0,&quot;id&quot;:&quot;h-div_with_one_comment-list_end-2020-01-22T23:19:00.000Z&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment-list_end&quot;],&quot;headingLevel&quot;:2,&quot;placeholderHeading&quot;:false}" class="ext-discussiontools-init-section"><span data-mw-comment-start="" id="h-div_with_one_comment-list_end-2020-01-22T23:19:00.000Z"></span>div with one comment, ends with a list<span data-mw-comment-end="h-div_with_one_comment-list_end-2020-01-22T23:19:00.000Z"></span><span class="ext-discussiontools-init-section-subscribe mw-editsection-like"><span class="ext-discussiontools-init-section-subscribe-bracket">[</span><a href="" class="ext-discussiontools-init-section-subscribe-link" role="button" tabindex="0" data-mw-comment-name="h-Matma_Rex-2020-01-22T23:19:00.000Z" title="(discussiontools-topicsubscription-button-subscribe-tooltip)">(discussiontools-topicsubscription-button-subscribe)</a><span class="ext-discussiontools-init-section-subscribe-bracket">]</span></span></h2>
Fix replying outside wrappers for partially indented comments Top-level comments that start or end with a list (inconsistent indentation) would not have triggered the logic for detecting wrappers. Bug: T273692 Change-Id: Idcb4eed73e391f5f86eca2eb05cb3cea0d86f30a
2021-02-08 20:52:30 +00:00
<div style="background: beige;">
Give comments IDs so they can be scrolled to with hash links Bug: T265268 Change-Id: Idb985ed38bdb74e23cb7840899a61dc919f05f6f
2021-03-10 17:25:52 +00:00
<p><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment-list_end"></span>hello</p>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<ul><li>blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment-list_end&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment-list_end"></span></li></ul>
Fix replying outside wrappers for partially indented comments Top-level comments that start or end with a list (inconsistent indentation) would not have triggered the logic for detecting wrappers. Bug: T273692 Change-Id: Idcb4eed73e391f5f86eca2eb05cb3cea0d86f30a
2021-02-08 20:52:30 +00:00
</div>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<h2 id="div_with_one_comment-list_start" data-mw-comment="{&quot;type&quot;:&quot;heading&quot;,&quot;level&quot;:0,&quot;id&quot;:&quot;h-div_with_one_comment-list_start-2020-01-22T23:19:00.000Z&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment-list_start&quot;],&quot;headingLevel&quot;:2,&quot;placeholderHeading&quot;:false}" class="ext-discussiontools-init-section"><span data-mw-comment-start="" id="h-div_with_one_comment-list_start-2020-01-22T23:19:00.000Z"></span>div with one comment, starts with a list<span data-mw-comment-end="h-div_with_one_comment-list_start-2020-01-22T23:19:00.000Z"></span><span class="ext-discussiontools-init-section-subscribe mw-editsection-like"><span class="ext-discussiontools-init-section-subscribe-bracket">[</span><a href="" class="ext-discussiontools-init-section-subscribe-link" role="button" tabindex="0" data-mw-comment-name="h-Matma_Rex-2020-01-22T23:19:00.000Z" title="(discussiontools-topicsubscription-button-subscribe-tooltip)">(discussiontools-topicsubscription-button-subscribe)</a><span class="ext-discussiontools-init-section-subscribe-bracket">]</span></span></h2>
Fix replying outside wrappers for partially indented comments Top-level comments that start or end with a list (inconsistent indentation) would not have triggered the logic for detecting wrappers. Bug: T273692 Change-Id: Idcb4eed73e391f5f86eca2eb05cb3cea0d86f30a
2021-02-08 20:52:30 +00:00
<div style="background: beige;">
Give comments IDs so they can be scrolled to with hash links Bug: T265268 Change-Id: Idb985ed38bdb74e23cb7840899a61dc919f05f6f
2021-03-10 17:25:52 +00:00
<ul><li><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment-list_start"></span>hello</li></ul>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<p>blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment-list_start&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_one_comment-list_start"></span></p>
Fix replying outside wrappers for partially indented comments Top-level comments that start or end with a list (inconsistent indentation) would not have triggered the logic for detecting wrappers. Bug: T273692 Change-Id: Idcb4eed73e391f5f86eca2eb05cb3cea0d86f30a
2021-02-08 20:52:30 +00:00
</div>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<h2 id="table_with_one_comment" data-mw-comment="{&quot;type&quot;:&quot;heading&quot;,&quot;level&quot;:0,&quot;id&quot;:&quot;h-table_with_one_comment-2020-01-22T23:19:00.000Z&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-table_with_one_comment&quot;],&quot;headingLevel&quot;:2,&quot;placeholderHeading&quot;:false}" class="ext-discussiontools-init-section"><span data-mw-comment-start="" id="h-table_with_one_comment-2020-01-22T23:19:00.000Z"></span>table with one comment<span data-mw-comment-end="h-table_with_one_comment-2020-01-22T23:19:00.000Z"></span><span class="ext-discussiontools-init-section-subscribe mw-editsection-like"><span class="ext-discussiontools-init-section-subscribe-bracket">[</span><a href="" class="ext-discussiontools-init-section-subscribe-link" role="button" tabindex="0" data-mw-comment-name="h-Matma_Rex-2020-01-22T23:19:00.000Z" title="(discussiontools-topicsubscription-button-subscribe-tooltip)">(discussiontools-topicsubscription-button-subscribe)</a><span class="ext-discussiontools-init-section-subscribe-bracket">]</span></span></h2>
Give comments IDs so they can be scrolled to with hash links Bug: T265268 Change-Id: Idb985ed38bdb74e23cb7840899a61dc919f05f6f
2021-03-10 17:25:52 +00:00
<table border="1"><tbody><tr><td><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-table_with_one_comment"></span>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-table_with_one_comment&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-table_with_one_comment"></span>
Fix detecting decorative comment frames with whitespace As a result of 0fc71f60cd464e9cf54c4048985971953503ce51, "empty" text nodes (containing only whitespace) at the end of the comment may be inside the comment's range, and trying to ignore them caused the ranges not to match and the frame not to be detected. Now the code works whether they're inside the comment's range or not. Add a test case for wrapped discussion comments with HTML comments and with whitespace. Bug: T250126 Bug: T268407 Change-Id: I2217ff5a635fd1c9c9e803f46795b1bfb3d17535
2020-12-08 22:46:11 +00:00
</td></tr></tbody></table>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<h2 id="div_with_multiple_comments" data-mw-comment="{&quot;type&quot;:&quot;heading&quot;,&quot;level&quot;:0,&quot;id&quot;:&quot;h-div_with_multiple_comments-2020-01-22T23:19:00.000Z&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_multiple_comments&quot;,&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_multiple_comments-1&quot;],&quot;headingLevel&quot;:2,&quot;placeholderHeading&quot;:false}" class="ext-discussiontools-init-section"><span data-mw-comment-start="" id="h-div_with_multiple_comments-2020-01-22T23:19:00.000Z"></span>div with multiple comments<span data-mw-comment-end="h-div_with_multiple_comments-2020-01-22T23:19:00.000Z"></span><span class="ext-discussiontools-init-section-subscribe mw-editsection-like"><span class="ext-discussiontools-init-section-subscribe-bracket">[</span><a href="" class="ext-discussiontools-init-section-subscribe-link" role="button" tabindex="0" data-mw-comment-name="h-Matma_Rex-2020-01-22T23:19:00.000Z" title="(discussiontools-topicsubscription-button-subscribe-tooltip)">(discussiontools-topicsubscription-button-subscribe)</a><span class="ext-discussiontools-init-section-subscribe-bracket">]</span></span></h2>
Give comments IDs so they can be scrolled to with hash links Bug: T265268 Change-Id: Idb985ed38bdb74e23cb7840899a61dc919f05f6f
2021-03-10 17:25:52 +00:00
<div style="background: beige;"><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_multiple_comments"></span>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_multiple_comments-1"></span><span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_multiple_comments&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-Matma_Rex-2020-01-22T23:19:00.000Z&quot;],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_multiple_comments"></span>
<dl><dd><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-Matma_Rex-2020-01-22T23:19:00.000Z"></span>blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:2,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-Matma_Rex-2020-01-22T23:19:00.000Z&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-Matma_Rex-2020-01-22T23:19:00.000Z"></span></dd></dl>
blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_multiple_comments-1&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-div_with_multiple_comments-1"></span>
Fix detecting decorative comment frames with whitespace As a result of 0fc71f60cd464e9cf54c4048985971953503ce51, "empty" text nodes (containing only whitespace) at the end of the comment may be inside the comment's range, and trying to ignore them caused the ranges not to match and the frame not to be detected. Now the code works whether they're inside the comment's range or not. Add a test case for wrapped discussion comments with HTML comments and with whitespace. Bug: T250126 Bug: T268407 Change-Id: I2217ff5a635fd1c9c9e803f46795b1bfb3d17535
2020-12-08 22:46:11 +00:00
</div>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
<h2 id="table_with_multiple_comments" data-mw-comment="{&quot;type&quot;:&quot;heading&quot;,&quot;level&quot;:0,&quot;id&quot;:&quot;h-table_with_multiple_comments-2020-01-22T23:19:00.000Z&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-table_with_multiple_comments&quot;,&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-table_with_multiple_comments-1&quot;],&quot;headingLevel&quot;:2,&quot;placeholderHeading&quot;:false}" class="ext-discussiontools-init-section"><span data-mw-comment-start="" id="h-table_with_multiple_comments-2020-01-22T23:19:00.000Z"></span>table with multiple comments<span data-mw-comment-end="h-table_with_multiple_comments-2020-01-22T23:19:00.000Z"></span><span class="ext-discussiontools-init-section-subscribe mw-editsection-like"><span class="ext-discussiontools-init-section-subscribe-bracket">[</span><a href="" class="ext-discussiontools-init-section-subscribe-link" role="button" tabindex="0" data-mw-comment-name="h-Matma_Rex-2020-01-22T23:19:00.000Z" title="(discussiontools-topicsubscription-button-subscribe-tooltip)">(discussiontools-topicsubscription-button-subscribe)</a><span class="ext-discussiontools-init-section-subscribe-bracket">]</span></span></h2>
Give comments IDs so they can be scrolled to with hash links Bug: T265268 Change-Id: Idb985ed38bdb74e23cb7840899a61dc919f05f6f
2021-03-10 17:25:52 +00:00
<table border="1"><tbody><tr><td><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-table_with_multiple_comments"></span>
Use placeholder localisation messages in CommentFormatter tests Otherwise they will fail whenever translations are updated (and they are failing right now). Change-Id: I849c57b86d36fb6c7739cc31a74df741e08462f4
2021-06-02 19:42:45 +00:00
blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-table_with_multiple_comments-1"></span><span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-table_with_multiple_comments&quot;,&quot;replies&quot;:[&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-Matma_Rex-2020-01-22T23:19:00.000Z-1&quot;],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-table_with_multiple_comments"></span>
<dl><dd><span data-mw-comment-start="" id="c-Matma_Rex-2020-01-22T23:19:00.000Z-Matma_Rex-2020-01-22T23:19:00.000Z-1"></span>blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:2,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-Matma_Rex-2020-01-22T23:19:00.000Z-1&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-Matma_Rex-2020-01-22T23:19:00.000Z-1"></span></dd></dl>
blah blah <b><a href="/wiki/User:Matma_Rex" title="User:Matma Rex">Matma Rex</a> | <a href="/wiki/User_talk:Matma_Rex" title="User talk:Matma Rex">talk</a></b> 23:19, 22 January 2020 (UTC)<span class="ext-discussiontools-init-replylink-buttons"><span class="ext-discussiontools-init-replylink-bracket">[</span><a class="ext-discussiontools-init-replylink-reply" role="button" tabindex="0" data-mw-comment="{&quot;type&quot;:&quot;comment&quot;,&quot;level&quot;:1,&quot;id&quot;:&quot;c-Matma_Rex-2020-01-22T23:19:00.000Z-table_with_multiple_comments-1&quot;,&quot;replies&quot;:[],&quot;timestamp&quot;:&quot;2020-01-22T23:19:00.000Z&quot;,&quot;author&quot;:&quot;Matma Rex&quot;}" href="">(discussiontools-replylink)</a><span class="ext-discussiontools-init-replylink-bracket">]</span></span><span data-mw-comment-end="c-Matma_Rex-2020-01-22T23:19:00.000Z-table_with_multiple_comments-1"></span>
Fix detecting decorative comment frames with whitespace As a result of 0fc71f60cd464e9cf54c4048985971953503ce51, "empty" text nodes (containing only whitespace) at the end of the comment may be inside the comment's range, and trying to ignore them caused the ranges not to match and the frame not to be detected. Now the code works whether they're inside the comment's range or not. Add a test case for wrapped discussion comments with HTML comments and with whitespace. Bug: T250126 Bug: T268407 Change-Id: I2217ff5a635fd1c9c9e803f46795b1bfb3d17535
2020-12-08 22:46:11 +00:00
</td></tr></tbody></table>
Reference in a new issue Copy permalink