Lachlan Campbell
Lachlan Campbell
Thanks for the issue. We are working on this in https://github.com/system-ui/theme-ui/pull/1090, but it’s been a big project. Help appreciated!!
Appreciate the report! Is this causing issues in your project? Or more like a "oh that's funky" that's not meaningfully slowing it down?
This would be awesome!
Regarding the variants, Theme UI currently doesn't support combining multiple variants, which is why this isn't working. That work is happening in #1226. I believe that's what's causing this issue,...
Doesn’t fix this issue, but Select now supports custom arrows via #1232
Yeah, these components are definitely heading in the legacy direction with `aspect-ratio` being supported in all modern browsers. https://caniuse.com/mdn-css_properties_aspect-ratio
We are not planning to implement this, since the `aspectRatio` CSS property can be used with an array for responsiveness. Would welcome a PR if anyone cares strongly. Thank you!
Note that we also have a simpler [`useColorMode()` hook](https://theme-ui.com/hooks/#usecolormode), but agree this would be nice to have too!
@hasparus I'm not sure if this is possible with how our packages are set up; since `color-modes` is an optional package, how would we override [this line](https://github.com/system-ui/theme-ui/blob/develop/packages/css/src/index.ts#L361) if that context...
Great news @johno! Our latest `develop` release fixes this issue thanks to @hasparus. Just forked your test repo & updated deps, it's working: https://github.com/lachlanjc/repro-mdx-v2-theme-ui/commit/5990f8e1c093e27493777e66df9f080f6d2deb1e