Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 8 days ago

#61507 closed defect (bug) (fixed)

Twenty Nineteen: Fix font size and citation display for Pullquote block

Reported by: pitamdey Owned by: karmatosed
Priority: normal Milestone: 6.7
Component: Bundled Theme Version:
Severity: normal Keywords: has-screenshots has-patch commit
Cc: Focuses: css

Description

In Theme Twenty Nineteen, Pullquote Block Text Decoration is not reflecting in frontend for citation but working fine in editor side
I have attached a screenshot for reference

Attachments (5)

Screenshot 2024-06-26 at 3.24.27 PM.png​ (210.0 KB ) - added by pitamdey 2 years ago.
Editor View
Screenshot 2024-06-26 at 3.24.38 PM.png​ (145.4 KB ) - added by pitamdey 2 years ago.
Frontend View
61507.patch​ (603 bytes ) - added by viralsampat 2 years ago.
I have checked above mentioned issue and I have resolved it and added patch. Also, I have attached screenshot after resolved this issue.
pullquote-after-resolve-editor-side.png​ (420.3 KB ) - added by viralsampat 2 years ago.
After resolved issue Back-end:
pullquote-after-resolve-front-end-side.png​ (335.8 KB ) - added by viralsampat 2 years ago.
After resolved issue front-end:

Download all attachments as: .zip

Change History (14)

This ticket was mentioned in ​PR #6913 on ​WordPress/wordpress-develop by ​@narenin.


2 years ago
#2

  • Keywords has-patch added

@viralsampat
2 years ago

I have checked above mentioned issue and I have resolved it and added patch. Also, I have attached screenshot after resolved this issue.

@viralsampat
2 years ago

After resolved issue Back-end:

@viralsampat
2 years ago

After resolved issue front-end:

This ticket was mentioned in ​PR #6945 on ​WordPress/wordpress-develop by ​@sabernhardt.


2 years ago
#3

  • Restores the default Pullquote paragraph font size to 49.5 (2.25em * 22px), which has been multiplied by 1.5 since WordPress 6.1.
  • Sets the cite element to display: block for consistent spacing and text-decoration.

Trac 61507

​@sabernhardt commented on ​PR #6945:


2 years ago
#4

Before patch: front end has extra-large font size, and the Pullquote does not honor the underline setting on the citation.

https://github.com/WordPress/wordpress-develop/assets/17100257/ecb4014d-8922-4628-b3a2-8aa234886df6

With patch: front end would reset the block's font size so the paragraph is 2.25em again, and the citation includes the underline according to the block option.

https://github.com/WordPress/wordpress-develop/assets/17100257/565bbf14-39b1-485c-b6c6-57cdaae8a4ee

Editor with patch (which is the same as before):
https://github.com/WordPress/wordpress-develop/assets/17100257/ea48695a-e3d8-4ba0-8588-de974a59cd33

Spacing comparison for citations of different lengths:

  • Before patch, a one-line citation has more space above it than a citation that wraps to a second line.
  • With the patch, both one-line and two-line citations have the same amount of space between them and the Pullquote paragraph.

https://github.com/WordPress/wordpress-develop/assets/17100257/ba375daa-1d57-4588-8f31-d2000365a790

Full images with each text decoration option:

#5 @sabernhardt
2 years ago

  • Focuses css added
  • Keywords needs-testing added
  • Milestone Awaiting Review → Future Release
  • Summary Twenty Nineteen : Pullquote Block Text Decoration is not reflecting in frontend for citation → Twenty Nineteen: Fix font size and citation display for Pullquote block

The font size is not directly related to the display property, but I wanted both to be fixed together.

PR 6945 includes the Sass and the two compiled stylesheets.

#6 @karmatosed
2 years ago

  • Owner set to karmatosed
  • Status new → assigned

I am going to assign this to myself to test and see what can be done about looking to commit this. Thanks.

#7 @karmatosed
2 years ago

  • Resolution → fixed
  • Status assigned → closed

In 58630:

Twenty Nineteen: Fixes font size and citation display for Pullquote block.

The pullquote block text decoration was not the same front and within the editor. This resolves that and resets.

Props pitamdey, viralsampat, sabernhardt.
Fixes #61507.

#8 @karmatosed
2 years ago

  • Keywords commit added

#9 @sabernhardt
2 years ago

  • Keywords needs-testing removed
  • Milestone Future Release → 6.7

​@sabernhardt commented on ​PR #6913:


8 days ago
#10

Thanks for the pull request! This was fixed in r58630.

Note: See TracTickets for help on using tickets.