Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#29895 closed defect (bug) (fixed)

Avoid empty markup in comments

Reported by: obenland Owned by: ocean90
Priority: normal Milestone: 4.1
Component: Comments Version: 2.7
Severity: normal Keywords: has-patch twenty-fifteen
Cc: Focuses: template

Description

When comment_reply_link() doesn't display a link, like when comments are closed, both Walker_Comment::comment() and Walker_Comment::html5_comment() still display the empty wrapper div.

This can lead to design irregularities when margins or paddings are applied to that element, but shouldn't since the link to pad is non-existent.

Attachments (1)

29895.diff (1.4 KB ) - added by obenland 12 years ago.

Download all attachments as: .zip

Change History (6)

@obenland
12 years ago

#1 @obenland
12 years ago

  • Keywords twenty-fifteen added

Tagging with twenty-fifteen to keep track of it.

#2 @anonymized_8769252
12 years ago

Tested the patch, looks good to me.

#3 @SergeyBiryukov
12 years ago

  • Component GeneralComments
  • Focuses template added
  • Milestone Awaiting Review4.1

#4 @ocean90
12 years ago

  • Owner set to ocean90
  • Resolutionfixed
  • Status newclosed

In 29908:

Comments: Don't print an empty HTML markup when comment_reply_link() returns no link.

props obenland.
fixes #29895.

#5 @helen
11 years ago

#25203 was marked as a duplicate.

Note: See TracTickets for help on using tickets.