-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Add additional text formatting options to the site title block #31734
Add additional text formatting options to the site title block #31734
Conversation
Size Change: +2.8 kB (0%) Total Size: 1.32 MB
ℹ️ View Unchanged
|
Depending on what is decided for the post title block, I do not believe it is a good idea to expose the underline control to users. |
…tor" This reverts commit bc8e22e.
Based on #31623 (comment) I tend to agree that underline is not helpful for this block, and I also think there is no need for strikethrough in terms of the site title, so have simplified the PR for now to just add the Appearance typography options. |
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.
This tests well for me in both the block and site editors 👍
I think the call to only include the font appearance support was a good one. LGTM!
Description
Partly fixes #27830 There is currently no way to add inline html to the site title block due to the fact that the site title can only be saved as a plan string, and is used in multiple places, eg. browser tab titles.
Adding additional text formatting block supports at least gives additional control of the formatting of the complete string.
To Test
Site title
blockScreenshots
Before:
After: