OpenNeo/impress-2020
OpenNeo/impress-2020
1
0
Fork 0
Code Pull requests Activity

Move "Log out" button into the nav menu

Browse source 
When actually using mobile, I was like "oh, it's maybe not great that the menu button is tiny and the Log out button is huge". Awkward click targets!
This commit is contained in:
Emi Matchu 2021-01-18 13:51:52 -08:00
parent 6e33132881
commit 9e5dfd1c84
1 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/app/GlobalHeader.js
View file

@ -129,7 +129,7 @@ function UserNavBarSection() {
<NavLinksList> <NavLinksList>
{id && ( {id && (
<NavLinkItem as={Link} to={`/user/${id}/items`}> <NavLinkItem as={Link} to={`/user/${id}/items`}>
Items Lists
</NavLinkItem> </NavLinkItem>
)} )}
<NavLinkItem as={Link} to={`/your-outfits`}> <NavLinkItem as={Link} to={`/your-outfits`}>
@ -138,10 +138,12 @@ function UserNavBarSection() {
<NavLinkItem as={Link} to="/modeling"> <NavLinkItem as={Link} to="/modeling">
Modeling Modeling
</NavLinkItem> </NavLinkItem>
<NavLinkItem
onClick={() => logout({ returnTo: window.location.origin })}
>
Log out
</NavLinkItem>
</NavLinksList> </NavLinksList>
<NavButton onClick={() => logout({ returnTo: window.location.origin })}>
Log out
</NavButton>
</HStack> </HStack>
); );
} else { } else {