- This topic has 10 replies, 3 voices, and was last updated 1 year ago by GG.
August 28, 2019 at 8:32 am #995586GG
Hi Tom. I think it would make more sense to move the reply button in the comments box to below the comment. It sounds odd or out of place when using a screen reader in its current position because it shows before a comment is read. I read somewhere that GP 2.4 will rectify that?August 28, 2019 at 9:40 am #995636DavidStaffCustomer Support
yes, this topic shows a change has already been committed to GP 2.4 that will allow the reply to be moved:August 28, 2019 at 8:12 pm #995933GG
That is good! Let me play around with the code for now until the feature comes officially.
Edit: I found I had to remove this code for the reply button at the top to disappear. How will this be dealt with in GP 2.4 so as not to show two instances of “reply”?
<?php comment_reply_link( array_merge( $args, array( 'add_below' => 'div-comment', 'depth' => $depth, 'max_depth' => $args['max_depth'], 'before' => '<span class="reply">| ', 'after' => '</span>', ) ) ); ?>August 29, 2019 at 2:54 am #996132DavidStaffCustomer Support
Did you follow Tom’s instructions to just edit your parent theme comments.php and use the exact filter he provided to move the reply?
I have tested this on another build and there is no duplicate reply.August 29, 2019 at 7:10 am #996270GG
Yes, I did. I copied the whole comments.php code he provided but still, two instances of reply showed up
http://prntscr.com/oz6ukj.August 29, 2019 at 8:08 am #996426DavidStaffCustomer SupportAugust 29, 2019 at 8:32 am #996453GG
Check the original post.August 29, 2019 at 7:46 pm #996819TomLead DeveloperLead Developer
Right now you’d need to hide it with CSS. I may actually move it under the content by default in 2.4, as it does make more sense down there. I’ll play with it.
Thanks!August 29, 2019 at 9:38 pm #996856GG
You are welcome. Does that mean also we can do away with that function to move the location altogether with GP 2.4?August 30, 2019 at 8:56 am #997326TomLead DeveloperLead DeveloperAugust 30, 2019 at 7:40 pm #997590GG
Okay. Will be on the lookout for that.
- You must be logged in to reply to this topic.