WordPress.org

Make WordPress Core

Opened 10 days ago

Last modified 4 days ago

#53112 accepted defect (bug)

Twenty Nineteen - Pullquote block editor styling breaks with block alignment set

Reported by: devnel Owned by: sabernhardt
Milestone: 5.8 Priority: normal
Severity: normal Version: 5.7.1
Component: Bundled Theme Keywords: has-patch needs-testing
Focuses: css Cc:

Description

Steps to reproduce the behavior

  1. Activate Twenty Nineteen
  2. Add a pull-quote block to any page
  3. Apply "wide-align" or any other block alignment option
  4. See the issue

What I expected to happen

The editor styling shows by default with the theme font styles (italic etc.) just like without alignment.

What actually happened

The editor styling of the text in the block doesn't reflect the correct size and font-decoration. The styling is still displayed correctly on the front-end and without any alignment options active.

Tested with latest WordPress version installed and no plugins active.

Attachments (5)

no-alignment.png (101.7 KB) - added by devnel 10 days ago.
Correct styling with no alignment set
wide-aligned.png (100.4 KB) - added by devnel 10 days ago.
Styling missing with wide-align set
53112.patch (524 bytes) - added by sabernhardt 6 days ago.
adding wp-block-pullquote class to Twenty Nineteen editor styles
Twenty-Nineteen-pullquotes-before.png (32.6 KB) - added by sabernhardt 6 days ago.
each alignment option before patch
Twenty-Nineteen-pullquotes-after.png (35.0 KB) - added by sabernhardt 6 days ago.
after applying patch

Download all attachments as: .zip

Change History (9)

@devnel
10 days ago

Correct styling with no alignment set

@devnel
10 days ago

Styling missing with wide-align set

#1 @sabernhardt
7 days ago

  • Component changed from Themes to Bundled Theme
  • Focuses css added

@sabernhardt
6 days ago

adding wp-block-pullquote class to Twenty Nineteen editor styles

@sabernhardt
6 days ago

each alignment option before patch

@sabernhardt
6 days ago

after applying patch

#2 @sabernhardt
6 days ago

  • Keywords has-patch needs-testing added

@devnel Thanks for the report!

53112.patch sets all editor pullquote styles to put the quote in italics and remove all-caps from the citation with Twenty Nineteen.

I also noticed a discrepancy between pullquote block alignment options with the Twenty Twenty theme, now reported on #53158.

Last edited 5 days ago by sabernhardt (previous) (diff)

#3 @desrosj
4 days ago

  • Milestone changed from Awaiting Review to 5.8

Moving to 5.8 because it would be nice to fix as many editor bugs in the default themes as possible. But with the focus on full site editing, this would need someone to own.

#4 @sabernhardt
4 days ago

  • Owner set to sabernhardt
  • Status changed from new to accepted
Note: See TracTickets for help on using tickets.