-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Verse: Add missing typography supports #43569
Verse: Add missing typography supports #43569
Conversation
There isn't currently a control in the global styles UI for text decoration (in TypographyPanel), so I think we've been merging in the text decoration PRs intentionally without exposing it there? Since text decoration is likely to be less used, from memory it was argued somewhere (possibly on one of these many PRs 😅) that it's okay if it appears as optional under the ToolsPanel menu, but in global styles we might not want to display it prominently, however theme authors can still use it via |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested well for me:
✅ Could add text decoration to verse in block editor and was applied in editor and frontend
✅ Could add setting to theme.json and it was applied in editor and frontend
When the text-decoration support was originally implemented, it was requested for it not to be exposed in Global Styles. The thinking was that text-decoration wasn't something that should be encouraged to be set globally by users however we should still offer it to theme authors (via theme.json). In a recent audit of the Global Styles Typography panel, it still wasn't on the horizon for making a comeback. If it interests you, you can follow the current progress to refining that typography panel. |
Related:
What?
Adds missing typography supports to the Verse block.
Why?
How?
Testing Instructions
Example theme.json snippet:
Screenshots or screencast
Screen.Recording.2022-08-24.at.7.31.07.pm.mp4