Skip to content
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 fontStyle control to Site Tagline block #35507

Merged
merged 1 commit into from Oct 11, 2021

Conversation

kjellr
Copy link
Contributor

@kjellr kjellr commented Oct 11, 2021

This brings the block in sync with the Site Title block. Since they're often used in tandem, I think it makes sense to include this control visible for both.

Many of the Twenty Twenty-Two header mockups use an italic site tagline, but in trunk (where there's currently just a "Weight" dropdown) it's impossible to make that happen via the UI.

To test:

  • Add a Site Tagline block
  • Note that it has an Appearance item in the sidebar, with both font weight and font style options.

Screenshot:

(Note that the "Weight" control changes to an "Appearance" control on the right)

Before After
Screen Shot 2021-10-11 at 8 08 32 AM Screen Shot 2021-10-11 at 8 08 54 AM

@kjellr kjellr added [Type] Enhancement A suggestion for improvement. [Block] Site Tagline Affects the Site Tagline Block labels Oct 11, 2021
@kjellr kjellr requested a review from jasmussen October 11, 2021 12:10
"__experimentalTextTransform": true,
"__experimentalFontStyle": true,
"__experimentalFontWeight": true,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I moved this one down just so that the order these are listed is in sync with the Site Title block.

@kjellr kjellr self-assigned this Oct 11, 2021
Copy link
Contributor

@jasmussen jasmussen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Before:
before

After:
after

Yep, before it was font weight only, now it's also style. Go for it.

@kjellr kjellr merged commit ec140ae into trunk Oct 11, 2021
@kjellr kjellr deleted the add/font-appearance-to-site-tagline branch October 11, 2021 13:49
@github-actions github-actions bot added this to the Gutenberg 11.8 milestone Oct 11, 2021
@mtias mtias mentioned this pull request Oct 11, 2021
66 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Site Tagline Affects the Site Tagline Block [Type] Enhancement A suggestion for improvement.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants