WordPress.org
Get WordPress

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#51533 closed defect (bug) (fixed)

Remove the additional space in the comment date link

Reported by: poena's profile poena Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.6 Priority: normal
Severity: normal Version:
Component: Comments Keywords: has-patch
Focuses: Cc:

Description

In class-walker-comment.php on line 436, in the comment metadata for the comment date, there is a space between </time> and </a> that is causing an extra space at the end of the link.

https://core.trac.wordpress.org/browser/trunk/src/wp-includes/class-walker-comment.php#L436

Attachments (2)

comment-date.png​ (15.2 KB) - added by poena 4 years ago.
The extra space is visible at the end of the link
51533.patch​ (1.2 KB) - added by mukesh27 4 years ago.
Patch that fixed the issue

Download all attachments as: .zip

Change History (9)

@poena
4 years ago

The extra space is visible at the end of the link

#1 @SergeyBiryukov
4 years ago

  • Milestone changed from Awaiting Review to 5.6

@mukesh27
4 years ago

Patch that fixed the issue

#2 @mukesh27
4 years ago

  • Keywords has-patch added

#3 @SergeyBiryukov
4 years ago

  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#4 @SergeyBiryukov
4 years ago

In 49157:

Comments: Remove extra space from the comment link in Walker_Comment.

Props poena, mukesh27, SergeyBiryukov.
See #51533.

#5 @SergeyBiryukov
4 years ago

In 49158:

Twenty Twenty: Remove extra space from the comment link in TwentyTwenty_Walker_Comment.

See #51533.

#6 @SergeyBiryukov
4 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 49159:

Twenty Nineteen: Remove extra space from the comment link in TwentyNineteen_Walker_Comment.

Fixes #51533.

#7 @SergeyBiryukov
4 years ago

In 49160:

Comments: Further remove unnecessary context switch in Walker_Comment.

Follow-up to [49157].

See #51533.

Note: See TracTickets for help on using tickets.