diff --git a/site/src/AppRouter.tsx b/site/src/AppRouter.tsx index bbcd2a7fd1..51a6895715 100644 --- a/site/src/AppRouter.tsx +++ b/site/src/AppRouter.tsx @@ -1,5 +1,5 @@ import React from "react" -import { Navigate, Route, Routes } from "react-router-dom" +import { Route, Routes } from "react-router-dom" import { AuthAndNav, RequireAuth } from "./components" import { PreferencesLayout } from "./components/Preferences/Layout" import { IndexPage } from "./pages" @@ -73,7 +73,6 @@ export const AppRouter: React.FC = () => ( }> - } /> } /> } /> } /> diff --git a/site/src/components/Navbar/UserDropdown.test.tsx b/site/src/components/Navbar/UserDropdown.test.tsx index 0a058b45b9..44730fd31f 100644 --- a/site/src/components/Navbar/UserDropdown.test.tsx +++ b/site/src/components/Navbar/UserDropdown.test.tsx @@ -50,6 +50,6 @@ describe("UserDropdown", () => { throw new Error("Anchor tag not found for the account menu item") } - expect(link.getAttribute("href")).toBe("/preferences") + expect(link.getAttribute("href")).toBe("/preferences/account") }) }) diff --git a/site/src/components/Navbar/UserDropdown.tsx b/site/src/components/Navbar/UserDropdown.tsx index 5b9d22b2ac..4b8bce5b2d 100644 --- a/site/src/components/Navbar/UserDropdown.tsx +++ b/site/src/components/Navbar/UserDropdown.tsx @@ -28,8 +28,8 @@ export interface UserDropdownProps { export const UserDropdown: React.FC = ({ user, onSignOut }: UserDropdownProps) => { const styles = useStyles() - const [anchorEl, setAnchorEl] = useState() + const handleDropdownClick = (ev: React.MouseEvent): void => { setAnchorEl(ev.currentTarget) } @@ -39,20 +39,18 @@ export const UserDropdown: React.FC = ({ user, onSignOut }: U return ( <> -
- -
- - - - {anchorEl ? ( - - ) : ( - - )} -
-
-
+ +
+ + + + {anchorEl ? ( + + ) : ( + + )} +
+
= ({ user, onSignOut }: U - - + + @@ -85,7 +83,7 @@ export const UserDropdown: React.FC = ({ user, onSignOut }: U - +