themes icon indicating copy to clipboard operation
themes copied to clipboard

Global Styles - Heading Block Typography Font changes are inconsistent across themes

Open cuemarie opened this issue 2 years ago • 10 comments

Quick summary

The Heading block's default font family is impacted incosistently across themes using Global Styles.

In many themes, including Twenty Twenty Three, the heading block's default fonts are set from Styles > Typography > Headings > FONT.

Some themes, though, aren't impacted by this change at all, and instead must set the default font for headings under Styles > Blocks > Heading > Typography > FONT.

In the comments of a similar issue, @BogdanUngureanu noted this regarding Yuga:

In the case of Yugo, we've noticed that it's caused because the theme styles the core/heading element with a font. Because of this Global Styles (font family in this case) defined in Typography > Headings are overridden. In order to style these elements from Global Styles, you'll have to go to Global Styles > Blocks > Heading > Typography - https://github.com/Automattic/wp-calypso/issues/71641#issuecomment-1441688361

In response to that report, a PR was merged that fixed this issue in Masu, for reference:

  • https://github.com/Automattic/themes/pull/6885

Steps to reproduce

  1. On a WordPress.com site, activate theme Russell, Yuga, or Tsubaki
  2. Open the Site Editor, and add Heading blocks to a template of your choice
  3. Open Styles > Typography > Headings and change the Font family.
    • Observe the heading blocks on the template do not change font.
  4. Now open Styles > Blocks > Heading > Typography and change the font family there
    • Observe the heading blocks on the template respond to the font change.

What you expected to happen

For Styles > Typography > Headings > FONT to successfully set a default font for Heading blocks.

Twenty Twenty Three Example

https://github.com/Automattic/wp-calypso/assets/27249804/0ec1c42c-22d7-4b89-8b2e-598f2814a826

What actually happened

Nothing happens. Instead, the heading blocks respond to changes made under Styles > Blocks > Heading > Typography > FONT.

Russell Theme

https://github.com/Automattic/wp-calypso/assets/27249804/0ca0d762-4cbe-4198-a34e-9937e9dca99a

Tsubaki Theme

https://github.com/Automattic/wp-calypso/assets/27249804/e7f74921-e28d-4953-af3b-7cf5348fa4c9

Impact

Some (< 50%)

Available workarounds?

Yes, easy to implement

Platform (Simple and/or Atomic)

Simple, Atomic

Logs or notes

It was suggested that all Blockbase themes may be impacted, so I also tested the community theme Blockbase on an Atomic site, and this theme did not replicate the discrepancy - heading default fonts could be set under Styles > Typography > Headings > FONT (as well as under the Block specific level).

https://github.com/Automattic/wp-calypso/assets/27249804/7cce7e6c-cf16-4138-a55d-747006dc5b12

cuemarie avatar Jun 21 '23 22:06 cuemarie

Meraki and Byrne were mentioned in the comments of the above issues as also suffering from this problem.

cuemarie avatar Jun 21 '23 23:06 cuemarie

📌 ACTIONS

  • So far, this has been addressed previously at the theme level, so I've transferred to Themes.
  • That said, CC-ing @Automattic/t-rex here too, in case there's anything outside the theme-specific level that can be done to address this.

cuemarie avatar Jun 21 '23 23:06 cuemarie

Notified Themes folks here: p1687389660658389-slack-C029FM1EH

cuemarie avatar Jun 21 '23 23:06 cuemarie

Hey folks! The fix that was created for Yuga look ready to deploy, just needs a helping hand: https://github.com/Automattic/theme-yuga/pull/8#issuecomment-1608290279

cuemarie avatar Jun 28 '23 01:06 cuemarie

I got another one: 6489125-zd. Theme is munchies

renata-franco avatar Jul 04 '23 17:07 renata-franco

Support References

This comment is automatically generated. Please do not edit it.

  • [ ] 6489125-zen
  • [ ] https://wordpress.com/forums/topic/how-to-change-defaults-for-h1-h2-h3-and-h3-styles
  • [ ] 6876915-zen
  • [ ] 7093987-zen
  • [ ] 8126714-zen

github-actions[bot] avatar Jul 04 '23 17:07 github-actions[bot]

Another report here: https://wordpress.com/forums/topic/how-to-change-defaults-for-h1-h2-h3-and-h3-styles/?view=all#post-3976558

The theme in use is Creatio, and in Styles > Colours > H4 > a text colour is set, but this does not reflect throughout the site.

aleone89 avatar Aug 01 '23 08:08 aleone89

6876915-zd

Theme= Tenaz, Issue is with post titles.

dolgelukkig avatar Sep 17 '23 08:09 dolgelukkig

7093987-zen

+1 - Creatio theme

aetta avatar Sep 28 '23 20:09 aetta

8126714-zd-a8c

An instance with the Twenty Twenty-Four theme; had to update the Heading block

ClassicRKR27 avatar Apr 30 '24 17:04 ClassicRKR27

This issue feels overly broad. It seems like the best path forward would be to open individual issues for each theme. Otherwise the scope of this issue is WAY to broad for an individual engineer to realistically pick up during a groundskeeper rotation.

I'm going to close this larger issue. I propose creating individual issues for each theme moving forward.

davemart-in avatar Oct 16 '24 15:10 davemart-in