-
Notifications
You must be signed in to change notification settings - Fork 360
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
Archeo: Try using button as an element #5849
Conversation
We can merge this when 13.4 is live |
@scruffian I added a few changes to make the buttons look like they should. The padding control for the search block button can only be done via the elements API or CSS, since the search block padding is not affecting the button, but the whole block. By assigning a padding to the button element, the outline variation is always bigger than the regular button is, so that required extra CSS to fix. What do you think? If you agree with that, I'll go ahead and replicate this on the rest of the themes. |
I'd rather we try to find ways to remove the CSS altogether. Are there changes we can make to Gutenberg so that we don't need it? |
if we could target block variations from theme.json... Can't think of anything else |
WordPress/gutenberg#27476 I believe would be the issue for this |
Ok that looks like it will be a while away yet then, but moving closer :) For now this seems fine... |
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.
Alright then! I'll approve this and bring this in once dotcom gets the GB change. I'll work on a PR for Blockbase next.
Ok this one is not done, it looks like we didn't remove the file button CSS since WordPress/gutenberg#41239 was merged, but that has bugs too. |
the bugs should be fixed in WordPress/gutenberg#41393 That makes 13.5 our merging target for this PR |
We should make a quick test on dotcom but this should be mergeable now |
Tested on wpcom. |
This is useful for testing https://github.com/WordPress/gutenberg/pull/40260/files
Things to note:
styles > blocks > core/button
tostyles > elements > button
.