Make WordPress Core

Opened 3 years ago

Closed 2 years ago

#55931 closed defect (bug) (fixed)

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

Reported by: kajalgohel's profile kajalgohel Owned by: audrasjb's profile audrasjb
Milestone: 6.1 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-screenshots has-patch
Focuses: ui, css Cc:

Description

In Twenty Twenty Theme, when we add Quote block in editor side, We can see that "Add citation" text is seen with Italic font-style. But when we see the same Quote block at front side, "Add citation" text is seen with normal font-style.

Steps to replicate:
1: Activate the Twenty Twenty Theme
2: Add Quote block
3: Add text in "Add citation" section
4: Save Page/Post
5: View the page/post at front side

For better understanding I provide video attachment.

Thanks

Attachments (3)

CleanShot 2022-06-06 at 14.32.25.mp4 (5.1 MB) - added by kajalgohel 3 years ago.
#55931.patch (513 bytes) - added by kajalgohel 3 years ago.
patch added for normal font-style in editor side.
#55931.2.patch (1.0 KB) - added by kajalgohel 3 years ago.
patch added in RTL stylesheet.

Change History (20)

#1 @audrasjb
3 years ago

  • Milestone changed from Awaiting Review to 6.1
  • Version 6.0 deleted

#2 @audrasjb
3 years ago

  • Keywords has-screenshots added

Thank you for spotting this, reporting the ticket and adding a video recording of the issue :)

This ticket was mentioned in PR #2788 on WordPress/wordpress-develop by thejaydip.


3 years ago
#3

  • Keywords has-patch added; needs-patch removed

Fixed.
Ref #55931

#5 @whaze
3 years ago

hi, i add a PR, i modified the editor style

#6 @mukesh27
3 years ago

  • Keywords needs-refresh added; has-patch removed

Hi there!

Thanks for the patch!

The Quote block does not use <footer> tag for the Add citation can you please check it?

@kajalgohel
3 years ago

patch added for normal font-style in editor side.

#7 @kajalgohel
3 years ago

  • Keywords has-patch added

#8 @audrasjb
3 years ago

  • Owner set to audrasjb
  • Status changed from new to reviewing

Thanks everyone.
To sum-up the current status of the ticket:

  • PR2788 by @iamjaydip contains unwanted changes.
  • PR2789 by @whaze should target cite instead of footer.
  • 55931.patch by @kajalgohel needs to implement the change also on the RTL stylesheet.

:)

#9 @mukesh27
3 years ago

#55934 was marked as a duplicate.

@kajalgohel
3 years ago

patch added in RTL stylesheet.

#10 @kajalgohel
3 years ago

Hello @audrasjb

Thanks for the update. I have added css changes in both the stylesheet.

Thanks.

#11 @whaze
3 years ago

thx @audrasjb
i updated the PR

#12 @audrasjb
3 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 53470:

Twenty-Twenty: Improve Quote block style consistency between front and back-end.

This changeset updates the font-style value of the Quote block in the block editor to ensure styles are consistent between the editor and the front-end.

Props kajalgohel, iamjaydip, audrasjb, whaze, mukesh27.
Fixes #55931.

#15 @bhrugesh12
3 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Hello @audrasjb,
I've checked twenty twenty theme with previous version of WordPress(5.9.3) and style of cite was italic. So we need to improve it as italic.

#16 @audrasjb
3 years ago

  • Keywords needs-refresh removed

@bhrugesh12 using github blame, it appears the cite font-style value has always been normal in the front-end, so we'll want the editor view to be consistent with the font-end.
https://github.com/WordPress/wordpress-develop/blame/trunk/src/wp-content/themes/twentytwenty/style.css#L558

#17 @sabernhardt
2 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

I'll close this because r53470 fixes the bug in the block editor. The cite element not only had normal font-style in style.css since initial commit, but the classic editor styles have matched the front end since before Twenty Twenty became available.

#56557 suggests changing the styles for a different use of the cite element. Discussion about possible style improvements can continue there.

Note: See TracTickets for help on using tickets.