Make WordPress Core

Opened 23 months ago

Last modified 23 months ago

#55934 reopened defect (bug)

Twenty Twenty: "Add citation" text font-weight issue in Quote block

Reported by: kmadhak's profile kmadhak Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 5.3
Component: Bundled Theme Keywords: needs-patch
Focuses: ui, css Cc:

Description

In Twenty Twenty Theme, when we add a text on "Add citation" in Quote block and make the text as italic then in editor side it is visible as italic without font-weight. But when we see the same formatted text at front side, then it is with italic and having font-weight as bolder which is different from editor side.

Steps to replicate:
1: Activate the Twenty Twenty Theme
2: Add Quote block
3: Add text in "Add citation" section
4: Select the added text and make them as italic
5: Save Page/Post
6: View the page/post at front side

For better understanding I provide video attachment.

Thank You.

Attachments (3)

Untitled.mp4 (8.0 MB) - added by kmadhak 23 months ago.
font-weight.png (137.3 KB) - added by mukesh27 23 months ago.
#55934.patch (353 bytes) - added by kajalgohel 23 months ago.
patch added for font-weight in front side.

Change History (14)

@kmadhak
23 months ago

#1 @mukesh27
23 months ago

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

Duplicate of #55931.

#2 @kmadhak
23 months ago

  • Resolution duplicate deleted
  • Status changed from closed to reopened

Hello @mukesh27,

As we can see that, the ticket which you have mentioned as main ticket (https://core.trac.wordpress.org/ticket/55931) having font-style issue, which is totally different from the issue which I have reported here that is of font-weight issue, please check it once and let me know if there is anything.

Thank you.

#3 @mukesh27
23 months ago

@kmadhak Thanks for reopen. Yes it's different ticket.

As i see in my test the font weight for same 600

#4 follow-up: @kmadhak
23 months ago

Hello @mukesh27,

I have attached the video and provided the steps which help you to reproduce the issue, please find the attached video with this ticket and you can refer the steps which I have mentioned in description above.

Thank you.

#5 in reply to: ↑ 4 @hasanuzzamanshamim
23 months ago

Replying to kmadhak:

I tested on my local from the trunk and everything is working as expected.

https://prnt.sc/dOjD8xIa2OOb

#6 follow-up: @kmadhak
23 months ago

Hello @hasanuzzamanshamim,

As I have seen your screenshot, I guess you are using Twenty Twenty Two Theme, but you can regenerate this issue in Twenty Twenty Theme which I have mentioned in my description and title as well in this ticket.

Thank you.

#7 in reply to: ↑ 6 @hasanuzzamanshamim
23 months ago

Replying to kmadhak:

Oops! Thanks for pointing it out, Now I got the issue.

#8 @hasanuzzamanshamim
23 months ago

Twenty Twenty's style.css has

cite em,
cite i {
   font-weight: bolder;
}

also on the style-rtl.css file needs to fix.

@kajalgohel
23 months ago

patch added for font-weight in front side.

#9 @kajalgohel
23 months ago

  • Keywords has-patch added; needs-patch removed

#10 @sabernhardt
23 months ago

  • Keywords needs-patch added; has-patch removed
  • Milestone set to Awaiting Review
  • Version changed from 6.0 to 5.3

Making the font-weight bolder on the front end was a clear design decision from the beginning of the theme. People probably should expect that to remain the same, especially if they do not use the block editor.

Even so, the editor would need to reflect the bolder style.

#11 @kajalgohel
23 months ago

Hello @sabernhardt

As per user point of view and for consistency, editor and front side should be similar.
I think whatever changes we are performing at the editor side, it should reflect on the front side.

Thanks

Note: See TracTickets for help on using tickets.