Button states: outline button should have dark background on hover

[madhusudhand]

Following button states should match with figma.

Current:

Hover: (outline button background)

Focus: (dashed outline for both buttons)

Expected:

[carolinan]

There is no way to target the outline style variation using theme.json.

[mikachan]

There is no way to target the outline style variation using theme.json.

Yeah... we'd need to use CSS. I may create a label for these kinds of issues, so we can decide on how much CSS we want to add (if any).

Is there a Gutenberg issue for styling outline buttons via theme.json? Maybe this one? WordPress/gutenberg#42794

[MaggieCabrera]

I made a PR for this: WordPress/gutenberg#43526

[mikachan]

Thanks @MaggieCabrera! It's actually the outline button variation we need to style with the above, but allowing CSS outlines to be controlled via theme.json is a great addition for style variation submissions.

[MaggieCabrera]

Thanks @MaggieCabrera! It's actually the outline button variation we need to style with the above, but allowing CSS outlines to be controlled via theme.json is a great addition for style variation submissions.

ugh, goes to show what happens when you read GH issues too early in the morning! I still think my PR is a good one to have. The outline variations one is a much harder one and I don't think we can make that one on time given the work that needs to be done to block styles variations, sadly

[carolinan]

Once there is a UI for borders for buttons, I think the outline style should be removed from core, because the settings will make the block more flexible.

[MaggieCabrera]

Once there is a UI for borders for buttons, I think the outline style should be removed form core, because the settings will make the block more flexible.

That will work for this specific variation but not for others (think of maybe the dotted separator?) so there's value on being able to control these via theme.json. I'm also keen on having the ability to select a variation as default that is not the current default one.

[carolinan]

Yeah, selecting a different default used to be possible with the ui, but it was removed.

[mikachan]

Closing this, as unfortunately, it's not currently possible to style outline buttons using theme.json.