Make WordPress Core

Opened 3 months ago

Last modified 7 weeks ago

#59334 new defect (bug)

In Theme Twenty Nineteen, Twenty seventeen & Twenty Twenty-Two Comment section Reply and cancel reply has no space between them

Reported by: pitamdey's profile pitamdey Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-screenshots needs-testing-info
Focuses: ui, css Cc:

Description

In Theme Twenty Nineteen, Twenty seventeen & Twenty Twenty-Two, in the comment section on clicking reply the reply to user and cancel reply have no space between them
Reference Video : https://drive.google.com/file/d/1YBcQquFQdZz7CpUZs3gzYBfK869CTWDB/view

Attachments (6)

2017.png (122.5 KB) - added by pitamdey 3 months ago.
Twenty Seventeen
2019.png (144.2 KB) - added by pitamdey 3 months ago.
Twenty Nineteen
2022.png (159.4 KB) - added by pitamdey 3 months ago.
Twenty Twenty-Two
59334.patch (1.3 KB) - added by pitamdey 3 months ago.
After applying this solution the issue is resolved
59334.2.patch (2.9 KB) - added by himshekhar07 3 months ago.
Patch added
59334.3.patch (1.4 KB) - added by nidhidhandhukiya 3 months ago.

Download all attachments as: .zip

Change History (9)

@pitamdey
3 months ago

Twenty Seventeen

@pitamdey
3 months ago

Twenty Nineteen

@pitamdey
3 months ago

Twenty Twenty-Two

@pitamdey
3 months ago

After applying this solution the issue is resolved

@himshekhar07
3 months ago

Patch added

#1 @sabernhardt
3 months ago

  • Component changed from Comments to Bundled Theme

This might be fixable by adding a space in the comment reply JS instead (#51589), but you proposed changing theme styles.

Notes:

  • Twenty Seventeen uses em measurements for most margins. Its margin for could go in the comment-reply area of the stylesheet.
  • Twenty Nineteen likewise could use 0.5em (or 0.5rem) as a margin. That theme's styles need to be edited in SCSS and then compiled.
  • Twenty Twenty-Two only needs a correction in the legacy version of the Comments block; the newer block adds a margin of 0.5em. To avoid RTL issues, we could try a pseudo-element before the small element.
    .wp-block-post-comments .comment-reply-title > small::before {
      content: "";
      display: inline-block;
      width: 0.5em;
      height: 1em;
    }
    

#2 @poena
2 months ago

  • Keywords needs-testing-info added

Hi,
@pitamdey can you please clarify if you mean the comments block, or the theme's default comment area?

It would be helpful to add testing instructions, both for reproducing the problem and for testing the patch.

#3 @pitamdey
7 weeks ago

Hi, @poena... Its the theme's default comment area.

Last edited 7 weeks ago by pitamdey (previous) (diff)
Note: See TracTickets for help on using tickets.