Make WordPress Core

Opened 7 weeks ago

Closed 6 weeks ago

Last modified 2 weeks ago

#62689 closed defect (bug) (worksforme)

Twenty Nineteen & Twenty Twenty: Comments block CSS issues

Reported by: viralsampat's profile viralsampat Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: close
Focuses: css Cc:

Description

Hello team,

I have checked "Comment Block: and found that some spacing issue and background color issue into the author section.

Twenty Nineteen: I have checked and found some space into the author section on the editor site, But when we check into the front-end, There is no space.

I think that it should be same on both sides(Editor & Front).

Twenty Twenty: I have checked and found some space into the author section on the editor site, But when we check into the front-end, There is no space.

Also, one more thing that I notice in twenty twenty theme is there is no background color into the "Reply" on the editor site. But, When we check into the front-end, There is background color for "Reply".

For better understanding here I have provided screenshots for both twenty nineteen & twenty twenty theme.

Attachments (5)

twenty-nineteen-comment-block-editor-side.png (184.3 KB) - added by viralsampat 7 weeks ago.
Twenty nineteen back end
twenty-nineteen-comment-block-front-side.png (243.2 KB) - added by viralsampat 7 weeks ago.
Twenty nineteen front-end
twenty-twenty-comment-block-editor-side.png (192.9 KB) - added by viralsampat 7 weeks ago.
Twenty Twenty backend
twenty-twenty-comment-block-front-side.png (193.8 KB) - added by viralsampat 7 weeks ago.
Twenty Twenty Front-end
twenty-twenty-comment-block-front-side.2.png (193.8 KB) - added by viralsampat 7 weeks ago.
Twenty Twenty Front-end

Download all attachments as: .zip

Change History (10)

@viralsampat
7 weeks ago

Twenty nineteen back end

@viralsampat
7 weeks ago

Twenty nineteen front-end

@viralsampat
7 weeks ago

Twenty Twenty backend

@viralsampat
7 weeks ago

Twenty Twenty Front-end

@viralsampat
7 weeks ago

Twenty Twenty Front-end

#1 @parthvataliya
7 weeks ago

  • Keywords has-testing-info added

Reproduction Report

Description

This report validates whether the issue can be reproduced.

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.26
  • Server: nginx/1.27.3
  • Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.26)
  • Browser: Chrome 131.0.0.0
  • OS: Linux
  • Theme: Twenty Twenty 2.8
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.2.0

Actual Results

  1. ✅ Error condition occurs (reproduced).

#2 @sabernhardt
7 weeks ago

  • Keywords close added; dev-feedback needs-testing has-testing-info removed
  • Summary changed from Twenty Nineteen & Twenty Twenty themes: The comment block having some CSS issue. to Twenty Nineteen & Twenty Twenty: Comments block CSS issues
  • Version 6.7.1 deleted

Related: #55744 and #62688

When a block creates duplicate IDs or other broken markup, broken styles can be a good thing.

#3 @ranafge
7 weeks ago

Test Report

Description

This report validates whether the issue can be reproduced.

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.26
  • Server: nginx/1.27.3
  • Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.26)
  • Browser: Edge 131.0.0.0
  • OS: Linux
  • Theme: Twenty Nineteen 3.0
  • MU Plugins: None activated

Actual Results

✅ Error condition occurs (reproduced).

#4 @karmatosed
6 weeks ago

  • Milestone Awaiting Review deleted
  • Resolution set to worksforme
  • Status changed from new to closed

This has been flagged to close and I tend to agree with that because it can be the case that as @sabernhardt has stated for duplicate IDs or broken markup, broken styles can be something good. With that in mind the testing is validation this does happen. I am going to close with working as expected. Thank you everyone for your collaboration and this doesn't mean discussion can't continue.

#5 @sabernhardt
2 weeks ago

#62820 was marked as a duplicate.

Note: See TracTickets for help on using tickets.