flatironinstitute / stan-playground

Run Stan models in the browser
Apache License 2.0
36 stars 1 forks source link

Use MUI theming for all colors #208

Closed WardBrian closed 3 months ago

WardBrian commented 3 months ago

This is the start of addressing #186.

Before we can really customize the color scheme, it seemed useful to make the existing colors consistent as much as possible. This PR moves everything I could think of over to using the MUI theming system for colors.

I tried to leave the visual appearance the same as much as possible, aside from some color changes:

image

An easy way to verify that this allows the intended customization is updating App.tsx to have:

const theme = createTheme({
  palette: {
    mode: "dark",
  },
});

image

magland commented 3 months ago

I didn't go through the code carefully, but I tried it out and it looks fantastic.

WardBrian commented 3 months ago

I spotted a couple more places I had missed (mostly in load/save windows) and did some code reorg on the stylized tables @jsoules

jsoules commented 3 months ago

I spotted a couple more places I had missed (mostly in load/save windows)

Nice catch! And simplifies things further.