Skip to content

Commit

Permalink
fix dropdownItem warning
Browse files Browse the repository at this point in the history
  • Loading branch information
SuperMitic committed Dec 11, 2024
1 parent e6e852f commit da1f474
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 6 deletions.
2 changes: 1 addition & 1 deletion client/components/maps/content.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ const MapPage: React.FC<ContentProps> = ({ events, friends }) => {
},
(error) => {
fetchUserPosition();
},
}
);
} else {
fetchUserPosition();
Expand Down
21 changes: 16 additions & 5 deletions client/components/navbar/user-dropdown.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -43,11 +43,15 @@ export const UserDropdown = () => {
const handleChange = useCallback((event: CustomEvent<UserChangeEvents>) => {
switch (event.type) {
case "emailUpdated":
setUser((prev) => ({ ...prev, email: event.detail.updatedEmail } as Person));
setUser(
(prev) => ({ ...prev, email: event.detail.updatedEmail } as Person)
);
break;
case "avatarUpdated":
console.log("Avatar updated:", event.detail.updatedAvatar);
setUser((prev) => ({ ...prev, avatar: event.detail.updatedAvatar } as Person));
setUser(
(prev) => ({ ...prev, avatar: event.detail.updatedAvatar } as Person)
);
break;
default:
console.error("Unknown event type:", event);
Expand All @@ -61,7 +65,10 @@ export const UserDropdown = () => {
window.addEventListener("avatarUpdated", handleChange as EventListener);
return () => {
window.removeEventListener("emailUpdated", handleChange as EventListener);
window.removeEventListener("avatarUpdated", handleChange as EventListener);
window.removeEventListener(
"avatarUpdated",
handleChange as EventListener
);
};
}, [fetchUser, handleChange]);

Expand Down Expand Up @@ -98,19 +105,23 @@ export const UserDropdown = () => {
<DropdownItem
key="profile"
className="flex flex-col justify-start w-full items-start"
textValue="Profile"
>
<p>Signed in as</p>
<p>{user?.email || "Loading..."}</p>
</DropdownItem>
<DropdownItem key="settings">My Settings</DropdownItem>
<DropdownItem key="settings" textValue="settings">
My Settings
</DropdownItem>
<DropdownItem
key="logout"
color="danger"
className="text-danger"
textValue="logout"
>
Log Out
</DropdownItem>
<DropdownItem key="switch">
<DropdownItem key="switch" textValue="Switch Theme">
<DarkModeSwitch />
</DropdownItem>
</DropdownMenu>
Expand Down

0 comments on commit da1f474

Please sign in to comment.