Make WordPress Core

Opened 2 years ago

Closed 10 months ago

#56427 closed defect (bug) (worksforme)

Twenty Twenty-Two: editor font weight for bold text in Button block does not match front

Reported by: varshil151189's profile varshil151189 Owned by:
Milestone: Priority: normal
Severity: normal Version: 6.0.1
Component: Bundled Theme Keywords: needs-patch has-screenshots close
Focuses: ui, css Cc:

Description

Hello Team,

For Twenty Twenty Two Theme, when we add Button block in editor side, and make the text bold at editor end the font-weight is coming 600 while at Frontend it is coming 700.

Steps to replicate:
1: Activate the Twenty Twenty-Two Theme
2: Add Button block
3: Add Text in Button Block
4: Make Button text Bold from Top Toolbar
5: View the page/post at editor side
6: Save Page/Post
7: View the page/post at front side

For Better understanding i am providing video attachment link

Link: https://share.cleanshot.com/CNLRh4J9ut4dcWypHHuT

Thank you.

Attachments (1)

56427.patch (278 bytes) - added by priyank9033 20 months ago.

Download all attachments as: .zip

Change History (10)

This ticket was mentioned in Slack in #core-test by ironprogrammer. View the logs.


2 years ago

#2 @shibleemehdi
2 years ago

  • Keywords has-screenshots needs-testing added

#3 @ironprogrammer
2 years ago

  • Keywords needs-testing-info added

Thank you for the report and video @varshil151189!

It would be helpful if other testers could also reproduce this issue.

#4 @shraboni
2 years ago

Reproduction Report

This report validates that the issue can be reproduced.

Environment

  • OS: Windows 11
  • Web Server: Nginx
  • PHP: 7.3.5, 7.4.1, 8.0.0
  • WordPress: 6.0.1, 6.0.2
  • Browser: Chrome 105.0.5195.102, Firefox 104.0,
  • Theme: Twenty Twenty-Two
  • Active Plugins:
  • Gutenberg 14.0.2
  • WordPress Beta Tester 3.2.2

Actual Results

  • ✅ Error condition occurs (reproduced).

Supplemental Artifacts

I've added an attachment here: https://d.pr/v/fsi3D0

Last edited 2 years ago by shraboni (previous) (diff)

#5 @ironprogrammer
2 years ago

  • Keywords needs-testing needs-testing-info removed

Thank you for adding the written testing instructions, @varshil151189!

And thanks for the repro test, @shraboni 🙌🏻 Will re-add needs-testing once a patch has been provided.

#6 @priyank9033
20 months ago

I checked this issue and I found font-weight: 600; which is require font-weight: 700; for bold and I updated that in the patch file.

@priyank9033
20 months ago

#7 @sabernhardt
20 months ago

  • Summary changed from Twenty Twenty Two: Font Weight Issue in Button Text to Twenty Twenty-Two: editor font weight for bold text in Button block does not match front

Admin pages have given strong elements a font-weight of 600 since [26072], and it should stay that way outside the editor canvas.

The Gutenberg plugin already switched to an iframe for block themes in the post editor. If that is merged into 6.3, it would prevent the admin styles from affecting post content.

For another possibility, the CSS reset could include font-weight: bold for any theme.

Related: #58383

#8 @karmatosed
10 months ago

  • Keywords close added

I am not sure this is an issue now as I do see both set to 700 using latest version. I am going to cautiously set the close keyword but happy to revert this if anyone can counter that decision. Thank you everyone.

#9 @karmatosed
10 months ago

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

Thank you everyone, I am going to close this now.

Note: See TracTickets for help on using tickets.