argyleink / open-props

CSS custom properties to help accelerate adaptive and consistent design.

Home Page:https://open-props.style

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

`<details>` & `<button>` need minor alignment

argyleink opened this issue · comments

they are close but not the same, should be closer probably? in style that is.

Maybe make <button> more like <summary>? https://codepen.io/hchiam/pen/dyZQbqb
Since <details> typically have a <summary>, then <button> should be styled more like <summary>?

awesome start @hchiam! ❤️
after studying your pen, i forked it and jotted down some code thoughts: https://codepen.io/argyleink/pen/qBVGOgJ

i think aligning to the button is best, but there's aspects of the button that dont need to be shared, like the hover effect. cant tell what styles should move up into normalize out of buttons or to keep in buttons and just duplicate some values..? hopefully my thoughts help clear up the intent too! i was comparing things to the button styles recently shipped as an extra bundle, and i totally didnt make that clear in the issue.

Screen Shot 2022-03-07 at 10 05 44 PM