dstrukt
dstrukt
Yeah I think you're right, let's move it to that repo!
I see, sounds good .. will leave this task open for tracking, and we'll reference and close when/if we address those spacings.
100% agree on adding 24px, I use that quite often, and would help with some of the more option-heavy settings screens, but for that specifically I will likely suggest we...
Following up on this: > Though we use Bootstraps spacing utility classes way more than ours, I think the best way forward is to extend our scale with the 24px/1.5rem...
Assuming we don't switch any of these colors depending on the mode... Which makes me wonder if we should kill the Light/Dark/Secondary variants, but if we did this, it begs...
Alternatively, if we go with the 200 variant background-color, full opacity. I don't like it *as* much, but this could work. A) Playing around with colors a bit more... B)...
Logically, this makes a lot of sense, and sounds like it's more flexible/error proofed than our current implementation. Would want to take a look at the UI to confirm, but...
Oh I see, yes, that sounds pretty close to how I imagined it originally reading the PR. Sounds like great change! cACK