Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

Try: add heading block style to paragraph #366

Closed
wants to merge 5 commits into from

Conversation

MaggieCabrera
Copy link
Collaborator

@MaggieCabrera MaggieCabrera commented Sep 14, 2023

This PR creates a paragraph style variation so that the font family values can be altered by theme.json for the style variations to actually work as intended.

Why we use the css property of theme.json for this: WordPress/gutenberg#49550

@MaggieCabrera MaggieCabrera marked this pull request as draft September 14, 2023 15:35
@MaggieCabrera MaggieCabrera marked this pull request as ready for review September 14, 2023 16:14
@carolinan
Copy link
Contributor

carolinan commented Sep 14, 2023

I think styling a paragraph as a heading -even only by the name - is a bad idea because it can give users the idea that headings and paragraphs are interchangeable.

We should not make it easier for the user to make mistakes that can cause problems both for SEO and accessibility.

I don't have a great alternative at the moment...
I think we could use a class name without the confusing block style variation, or, use a custom css property for the alternate font family in theme.json + theme style variations.-

@MaggieCabrera
Copy link
Collaborator Author

I agree, this was just an exploration, but I'm not a fan either

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants