Piotr Monwid-Olechnowicz comments

Results 140 comments of


Piotr Monwid-Olechnowicz

chore: Included githubactions in the dependabot config

Hey @naveensrinivasan. Thank you for the interest in Theme UI. I'm sorry to say that, but we're unfortunately not going to merge this. As a rule of thumb, we are...

Pass the current color mode to functions used in `sx`

Aren't all functions dependent on colors unsafe in SSR?

Theme UI provider import causes an ESM error when mdx-js/react is subsequently required

@web-programmer-here, use `--legacy-peer-deps` for now as a workaround until a new version of Theme UI is released.

docs: Specify MDX version to install

@john20xdoe Please use `--legacy-peer-deps` for now. MDX 1 doesn't depend on React 17, it was just written at the time React 17 was the newest version.

Improve Spinner performance

Thanks for the issue @lbebber! Great catch!

Document full .json for Components for more precise style overriding

> I spent many hours trying to figure out how to build upon the styling for Components I saw in the documentation. For example, https://theme-ui.com/components/radio . It was non-trivial to...

docs(color-modes): Add document about color value can be obj/arr/hex.

I think that part of the docs was more of _"How does Theme UI color modes logic work?"_ instead of _"How to define your color modes?_". We might need to...

style-guide: ColorPalette doesn't show color labels

@braaar TBH I'm not sure if it would build at all if you mean the deep import `../../color`. I see no reason not to import `{ getColor } from '@theme-ui/color'`...

colors from outside of theme are not recomputed on initial render after Gatsby SSR

Thanks for the issue and the reproduction @ehowey!

colors from outside of theme are not recomputed on initial render after Gatsby SSR

This is definitely something I broke while fixing color modes flash — previously we always started with default color mode and you could see the colors (except background and text)...