Changeset 49191 for branches/5.5

Timestamp:

10/18/2020 05:09:46 PM (6 years ago)

Author:

SergeyBiryukov

Message:

Comments: Hide the link to the initial parent comment in the reply heading when replying to another comment.

This ensures that the reply heading is updated correctly on single post URLs with the ?replytocom query argument.

Follow-up to [47506], [48876], [48904].

Props mailnew2ster, wpamitkumar, audrasjb, desrosj, SergeyBiryukov.
Merges [49187] to the 5.5 branch.
Fixes #51175.

Location:

branches/5.5

Files:

• . (modified) (1 prop)
• src/js/_enqueues/lib/comment-reply.js (modified) (3 diffs)

branches/5.5/src/js/_enqueues/lib/comment-reply.js

@@ -179 +179 @@
                 var replyHeadingElement  = getElementById( config.commentReplyTitleId );
                 var replyHeadingTextNode = replyHeadingElement && replyHeadingElement.firstChild;
+                var replyLinkToParent    = replyHeadingTextNode && replyHeadingTextNode.nextSibling;
 
                 if ( replyHeadingTextNode && replyHeadingTextNode.nodeType === Node.TEXT_NODE && headingText ) {
+                        if ( replyLinkToParent && 'A' === replyLinkToParent.nodeName && replyLinkToParent.id !== config.cancelReplyId ) {
+                                replyLinkToParent.style.display = '';
+                        }
+
                         replyHeadingTextNode.textContent = headingText;
                 }
@@ -320 +325 @@
                 var replyHeading         = getElementById( config.commentReplyTitleId );
                 var replyHeadingTextNode = replyHeading && replyHeading.firstChild;
+                var replyLinkToParent    = replyHeadingTextNode && replyHeadingTextNode.nextSibling;
 
                 if ( ! addBelowElement || ! respondElement || ! parentIdField ) {
@@ -343 +349 @@
 
                 if ( replyHeadingTextNode && replyHeadingTextNode.nodeType === Node.TEXT_NODE ) {
+                        if ( replyLinkToParent && 'A' === replyLinkToParent.nodeName && replyLinkToParent.id !== config.cancelReplyId ) {
+                                replyLinkToParent.style.display = 'none';
+                        }
+
                         replyHeadingTextNode.textContent = replyTo;
                 }