diff --git a/site/src/components/NavbarView/NavbarView.test.tsx b/site/src/components/NavbarView/NavbarView.test.tsx index 2b65a81b0026c..cae48b7a8ddcf 100644 --- a/site/src/components/NavbarView/NavbarView.test.tsx +++ b/site/src/components/NavbarView/NavbarView.test.tsx @@ -20,21 +20,6 @@ describe("NavbarView", () => { process.env = env }) - it("renders content", async () => { - // When - render( - , - ) - - // Then - await screen.findAllByText("Coder", { exact: false }) - }) - it("workspaces nav link has the correct href", async () => { render( > = ({ >
- {logo_url ? ( - Custom Logo - ) : ( - - )} +
+ {logo_url ? ( + Custom Logo + ) : ( + + )} +
> = ({ {logo_url ? ( Custom Logo ) : ( - + )} @@ -159,20 +161,13 @@ const useStyles = makeStyles((theme) => ({ root: { height: navHeight, background: theme.palette.background.paper, - "@media (display-mode: standalone)": { - borderTop: `1px solid ${theme.palette.divider}`, - }, borderBottom: `1px solid ${theme.palette.divider}`, - transition: "margin 150ms ease", }, wrapper: { position: "relative", display: "flex", justifyContent: "space-between", alignItems: "center", - margin: "0 auto", - maxWidth: containerWidth, - padding: `0 ${sidePadding}px`, [theme.breakpoints.up("md")]: { justifyContent: "flex-start", }, @@ -195,6 +190,7 @@ const useStyles = makeStyles((theme) => ({ }, }, profileButton: { + paddingRight: theme.spacing(2), [theme.breakpoints.up("md")]: { marginLeft: "auto", }, @@ -208,13 +204,19 @@ const useStyles = makeStyles((theme) => ({ alignItems: "center", display: "flex", height: navHeight, - paddingRight: theme.spacing(4), + color: theme.palette.text.primary, + padding: theme.spacing(2), // svg is for the Coder logo, img is for custom images "& svg, & img": { - width: 109, - maxHeight: `calc(${navHeight}px)`, + width: "100%", + height: "100%", + objectFit: "contain", }, }, + drawerLogo: { + padding: 0, + maxHeight: theme.spacing(5), + }, title: { flex: 1, textAlign: "center", @@ -238,26 +240,8 @@ const useStyles = makeStyles((theme) => ({ // NavLink adds this class when the current route matches. "&.active": { - position: "relative", color: theme.palette.text.primary, fontWeight: "bold", - - "&::before": { - content: `" "`, - left: 0, - width: 2, - height: "100%", - background: theme.palette.secondary.dark, - position: "absolute", - - [theme.breakpoints.up("md")]: { - bottom: 0, - left: theme.spacing(3), - width: `calc(100% - 2 * ${theme.spacing(3)}px)`, - right: theme.spacing(3), - height: 2, - }, - }, }, [theme.breakpoints.up("md")]: {