Skip to content

Commit b71a753

Browse files
authored
UX: Auth: Logout button is inaccessible (#1016)
1 parent 8e8ff06 commit b71a753

File tree

3 files changed

+3
-8
lines changed

3 files changed

+3
-8
lines changed

frontend/src/components/NavBar/UserInfo/UserInfo.styled.ts

-3
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,4 @@ export const Text = styled.div`
1717
&:hover {
1818
color: ${({ theme }) => theme.user.hoverColor};
1919
}
20-
}
2120
`;
22-
23-
export const LogoutLink = styled.a``;

frontend/src/components/NavBar/UserInfo/UserInfo.tsx

+1-3
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,7 @@ const UserInfo = () => {
1919
</S.Wrapper>
2020
}
2121
>
22-
<DropdownItem href="logout">
23-
<S.LogoutLink>Log out</S.LogoutLink>
24-
</DropdownItem>
22+
<DropdownItem href="/logout">Log out</DropdownItem>
2523
</Dropdown>
2624
) : null;
2725
};

frontend/src/components/common/Dropdown/DropdownItem.tsx

+2-2
Original file line numberDiff line numberDiff line change
@@ -15,12 +15,12 @@ const DropdownItem = React.forwardRef<unknown, DropdownItemProps>(
1515
const confirmation = useConfirm();
1616

1717
const handleClick = (e: ClickEvent) => {
18-
if (!onClick) return;
19-
2018
// eslint-disable-next-line no-param-reassign
2119
e.stopPropagation = true;
2220
e.syntheticEvent.stopPropagation();
2321

22+
if (!onClick) return;
23+
2424
if (confirm) {
2525
confirmation(confirm, onClick);
2626
} else {

0 commit comments

Comments
 (0)