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 | Owned by: | 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)
Change History (20)
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
This ticket was mentioned in PR #2789 on WordPress/wordpress-develop by whaze.
3 years ago
#4
Trac ticket: https://core.trac.wordpress.org/ticket/55931
#6
@
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?
#8
@
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 offooter
. - 55931.patch by @kajalgohel needs to implement the change also on the RTL stylesheet.
:)
#10
@
3 years ago
Hello @audrasjb
Thanks for the update. I have added css changes in both the stylesheet.
Thanks.
#15
@
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
@
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
@
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.
Thank you for spotting this, reporting the ticket and adding a video recording of the issue :)