Opened 5 years ago
Last modified 2 months ago
#51589 new defect (bug)
Missing whitespace between comment reply title and cancel comment reply link
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Awaiting Review | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Comments | Keywords: | reporter-feedback has-patch |
| Focuses: | Cc: |
Description
Hi,
There's a missing whitespace between the comment reply title and the cancel comment reply link.
This results in: "Reply to A WordPress CommenterCancel reply".
The cancel-comment-reply-link is wrapped in <small> tags.
In file comment-template there's a whitespace before the opening tag: 'cancel_reply_before' => ' <small>',
But somehow it's being ignored in comment form. Stripped perhaps? Was not able to find what causes this.
More info here:
https://wordpress.org/support/topic/cancel-comment-reply-link-in-comments-form/.
Change History (11)
#2
@
5 years ago
Hi David,
No, don’t think so.
Please check for example the TwentyNineteen theme. TwentyTwenty has some margin-left added to the link, so no visual issue with that one.
Guido
#3
@
5 years ago
The cancel reply link is placed within the title's before/after HTML, making it difficult to reorder elements with CSS as might be desired by some themes. The cancel reply link ought to be its own element outside of title HTML. In other words, move the echo $args['title_reply_after']; line up to before the echo $args['cancel_reply_before']; line.
This ticket was mentioned in PR #8419 on WordPress/wordpress-develop by Guido07111975.
11 months ago
#4
- Keywords has-patch added
Fix missing white space between comment reply title and cancel comment reply link.
Currently all themes use CSS to fix this.
There's a single white space added, but that's ignored: ' <small>'
When adding white space before ánd after the tag it will display properly: ' <small> '
As mentioned in this trac ticket.
This ticket was mentioned in PR #8423 on WordPress/wordpress-develop by Guido07111975.
11 months ago
#6
Fix missing white space between comment reply title and cancel comment reply link.
Currently all themes use CSS to fix this.
There's a single white space added, but that's ignored: ' <small>'
When adding white space before ánd after the tag it will display properly: ' <small> '
As mentioned in this trac ticket.
Guido07111975 commented on PR #8419:
11 months ago
#7
Closed PR because from wrong branch. Created new PR.
Guido07111975 commented on PR #8423:
6 months ago
#8
Revert previous commit and remove single white space before small tag, because it has no visible effect.
Maybe adding white space with CSS (such as now) instead of hard coded is sufficent?
Guido07111975 commented on PR #8423:
5 months ago
#9
Looks like wp_parse_args() strips white space ' ' but respects .
Guido07111975 commented on PR #8423:
5 months ago
#10
Reverted last commit.
Removed white space before the small tag, because it's stripped. So it has no added value.
I did not find a fix for this. Currently all themes use CSS to fix this, so maybe just leave it at that..
@Guido07111975 commented on PR #8423:
2 months ago
#11
Closed PR because I'm not able to fix this properly.
@Guido07111975 Do you have anything in your theme/plugins hooking in to
comment_form_defaults?