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

Block settings: Update variant of "Apply globally" Button component to secondary #61850

Merged
merged 3 commits into from
May 23, 2024

Conversation

amitraj2203
Copy link
Contributor

@amitraj2203 amitraj2203 commented May 22, 2024

What?

Fixes: #61834

Why?

The prominence of "Apply globally" button stands out even more when we have now the option to enable/disable overrides.

How?

This PR changes the variant of Apply globally button to secondary

Testing Instructions

  1. Activate any FSE theme. For example TT4
  2. Go to Appearance > Editor
  3. Go to Patterns > My Patterns and click on any pattern to edit it.
  4. On the right-hand side, go to the Advanced panel and see the Apply globally button at the bottom. (Make sure to change any styles so that the button comes in active state)

Screenshots

image

Copy link

github-actions bot commented May 22, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: amitraj2203 <amitraj2203@git.wordpress.org>
Co-authored-by: jasmussen <joen@git.wordpress.org>
Co-authored-by: t-hamano <wildworks@git.wordpress.org>
Co-authored-by: annezazu <annezazu@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@Mamaduka Mamaduka added [Type] Enhancement A suggestion for improvement. General Interface Parts of the UI which don't fall neatly under other labels. labels May 22, 2024
@Mamaduka Mamaduka requested a review from jasmussen May 22, 2024 15:04
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 one.

@t-hamano t-hamano self-requested a review May 23, 2024 11:29
Copy link
Contributor

@t-hamano t-hamano left a comment

Choose a reason for hiding this comment

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

LGTM !

@t-hamano t-hamano merged commit f49e0b5 into WordPress:trunk May 23, 2024
67 checks passed
@github-actions github-actions bot added this to the Gutenberg 18.5 milestone May 23, 2024
carstingaxion pushed a commit to carstingaxion/gutenberg that referenced this pull request Jun 4, 2024
…o secondary (WordPress#61850)

* Update variant of Apply globally Button component in PushChangesToGlobalStylesControl

* Add __next40pxDefaultSize prop to Apply globally Button

Co-authored-by: amitraj2203 <amitraj2203@git.wordpress.org>
Co-authored-by: jasmussen <joen@git.wordpress.org>
Co-authored-by: t-hamano <wildworks@git.wordpress.org>
Co-authored-by: annezazu <annezazu@git.wordpress.org>
patil-vipul pushed a commit to patil-vipul/gutenberg that referenced this pull request Jun 17, 2024
…o secondary (WordPress#61850)

* Update variant of Apply globally Button component in PushChangesToGlobalStylesControl

* Add __next40pxDefaultSize prop to Apply globally Button

Co-authored-by: amitraj2203 <amitraj2203@git.wordpress.org>
Co-authored-by: jasmussen <joen@git.wordpress.org>
Co-authored-by: t-hamano <wildworks@git.wordpress.org>
Co-authored-by: annezazu <annezazu@git.wordpress.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General Interface Parts of the UI which don't fall neatly under other labels. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Block settings: Change "Apply globally" to secondary button styling to lower prominence
4 participants