diff --git a/Client/src/Components/Breadcrumbs/index.jsx b/Client/src/Components/Breadcrumbs/index.jsx index 80fc4b3fe..0ebd8dab1 100644 --- a/Client/src/Components/Breadcrumbs/index.jsx +++ b/Client/src/Components/Breadcrumbs/index.jsx @@ -28,15 +28,16 @@ const Breadcrumbs = ({ list }) => { px={theme.spacing(2)} py={theme.spacing(3.5)} width="fit-content" - backgroundColor={ - theme.palette.secondary.main /* theme.palette.primary.lowContrast */ - } + backgroundColor={theme.palette.secondary.main} borderRadius={theme.shape.borderRadius} lineHeight="18px" sx={{ + "& .MuiBreadcrumbs-li a": { + transition: "background-color 0.2s ease-in-out, color 0.2s ease-in-out", + }, "& .MuiBreadcrumbs-li:not(:last-of-type):hover a": { - backgroundColor: theme.palette.primary.lowContrast, - opacity: 1, + backgroundColor: theme.palette.secondary.contrastText, + color: theme.palette.secondary.main, }, }} > diff --git a/Client/src/Components/Charts/BarChart/index.jsx b/Client/src/Components/Charts/BarChart/index.jsx index 9d487bf5f..5e01d74b8 100644 --- a/Client/src/Components/Charts/BarChart/index.jsx +++ b/Client/src/Components/Charts/BarChart/index.jsx @@ -39,6 +39,7 @@ const BarChart = ({ checks = [] }) => { > {checks.map((check, index) => check === "placeholder" ? ( + /* TODO what is the purpose of this box? */ { height={theme.spacing(4)} backgroundColor={ check.status - ? theme.palette.success.main - : theme.palette.error.contrastText + ? theme.palette.success.accent + : theme.palette.error.accent } sx={{ borderRadius: "50%" }} /> @@ -120,16 +121,18 @@ const BarChart = ({ checks = [] }) => { borderRadius: theme.shape.borderRadius, boxShadow: theme.shape.boxShadow, px: theme.spacing(4), - py: theme.spacing(2), + py: theme.spacing(3), }, "& .MuiTooltip-tooltip p": { + /* TODO Font size should point to theme */ fontSize: 12, - color: theme.palette.primary.contrastText.tertiary, + color: theme.palette.secondary.contrastText, fontWeight: 500, }, "& .MuiTooltip-tooltip span": { + /* TODO Font size should point to theme */ fontSize: 11, - color: theme.palette.primary.contrastText.tertiary, + color: theme.palette.secondary.contrastText, fontWeight: 600, }, }, @@ -143,13 +146,15 @@ const BarChart = ({ checks = [] }) => { backgroundColor={ check.status ? theme.palette.success.contrastText - : theme.palette.error.dark + : theme.palette.error.contrastText } sx={{ borderRadius: theme.spacing(1.5), + /* + TODO this doesn't seem to be used "&:hover > .MuiBox-root": { filter: "brightness(0.8)", - }, + }, */ }} > { width="100%" height={`${animate ? check.responseTime : 0}%`} backgroundColor={ - check.status ? theme.palette.success.main : theme.palette.error.main + check.status ? theme.palette.success.accent : theme.palette.error.accent } sx={{ borderRadius: theme.spacing(1.5), diff --git a/Client/src/Pages/Uptime/Home/UptimeTable/index.jsx b/Client/src/Pages/Uptime/Home/UptimeTable/index.jsx index 842e6c199..4853f5a94 100644 --- a/Client/src/Pages/Uptime/Home/UptimeTable/index.jsx +++ b/Client/src/Pages/Uptime/Home/UptimeTable/index.jsx @@ -178,7 +178,14 @@ const MonitorTable = ({ isAdmin, filter, setIsSearching, isSearching, handlePaus )} - +
{ sx={{ color: percentageColor, fontWeight: 500, + /* TODO point font weight to theme */ ml: "15px", }} > diff --git a/Client/src/Pages/Uptime/Home/index.jsx b/Client/src/Pages/Uptime/Home/index.jsx index 4fc57a7a0..1a60325f0 100644 --- a/Client/src/Pages/Uptime/Home/index.jsx +++ b/Client/src/Pages/Uptime/Home/index.jsx @@ -64,7 +64,14 @@ const UptimeMonitors = () => { onClick={() => { navigate("/uptime/create"); }} - sx={{ fontWeight: 500, whiteSpace: "nowrap" }} + sx={{ + fontWeight: 500, + whiteSpace: "nowrap", + /* + TODO Problem, could not make button mui work + backgroundColor: theme.palette.accent.main, + color: theme.palette.accent.contrastText, */ + }} > Create new diff --git a/Client/src/Utils/Theme/constants.js b/Client/src/Utils/Theme/constants.js index 933dd61d0..61c84f4fc 100644 --- a/Client/src/Utils/Theme/constants.js +++ b/Client/src/Utils/Theme/constants.js @@ -319,7 +319,7 @@ const newSemanticColors = { secondary: { main: { light: newColors.gray100, - dark: newColors.gray100, + dark: newColors.blueGray600, }, contrastText: { light: newColors.blueGray600, diff --git a/Client/src/Utils/Theme/globalTheme.js b/Client/src/Utils/Theme/globalTheme.js index 8719f8483..7d7a7ba0a 100644 --- a/Client/src/Utils/Theme/globalTheme.js +++ b/Client/src/Utils/Theme/globalTheme.js @@ -41,6 +41,7 @@ const baseTheme = (palette) => ({ defaultProps: { disableRipple: true, }, + styleOverrides: { root: ({ theme }) => ({ variants: [ @@ -61,6 +62,13 @@ const baseTheme = (palette) => ({ backgroundColor: theme.palette.secondary.main, }, }, + /* { + props: (props) => props.variant === "contained", + style: { + backgroundColor: `${theme.palette.accent.main} !important`, + }, + }, */ + { props: (props) => props.variant === "contained" && props.color === "secondary",