search

extendify / extendable

WordPress block theme for Full Site Editing (FSE) brought to you from the team at Extendify.
https://wordpress.org/themes/extendable
26 stars 1 forks source link

More style improvements #82

Closed richtabor closed 2 years ago

amjadr360 commented 2 years ago

Every thing else looks good to me except:

Piraeus variant: The button size is too large. image

Brisbane variant: Foreground color should be visible on secondary color. it's not a hard rule but a save option. We should remove the Group border radius.

image

I will write a PR for Button, alt button, and input box style fix. The default button should have extra +2px padding because the alt button has a 2px border by core. So both button variants can have the same height.

Thanks for standardizing the font size across the variants :)

richtabor commented 2 years ago

I will write a PR for Button, alt button, and input box style fix.

Feel free to add it to this PR even — that's fine by me.

amjadr360 commented 2 years ago

Feel free to add it to this PR even — that's fine by me.

Great! that's what I'm about to ask.