LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

fix sidebar for mobile

Browse source
This commit is contained in:
Sean Yesmunt 2020-12-17 13:43:47 -05:00
parent e21b5be6f6
commit b9f8fd73f1
2 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
ui/component/page/view.jsx
View file

@ -68,7 +68,8 @@ function Page(props: Props) {
if (isOnFilePage || isMediumScreen) {
setSidebarOpen(false);
}
}, [isOnFilePage, isMediumScreen, setSidebarOpen]);
// TODO: make sure setState callback for usePersistedState uses useCallback to it doesn't cause effect to re-run
}, [isOnFilePage, isMediumScreen]);
return (
<Fragment>

6
ui/component/sideNavigation/view.jsx
View file

@ -324,7 +324,7 @@ function SideNavigation(props: Props) {
// $FlowFixMe
const { hideForUnauth, ...passedProps } = linkProps;
return !email && linkProps.hideForUnauth && IS_WEB ? null : (
<li key={linkProps.icon}>
<li key={linkProps.link}>
<Button
{...passedProps}
label={__(linkProps.title)}
@ -390,7 +390,7 @@ function SideNavigation(props: Props) {
// $FlowFixMe
const { hideForUnauth, link, route, ...passedProps } = linkProps;
return !email && linkProps.hideForUnauth && IS_WEB ? null : (
<li key={linkProps.icon}>
<li key={route || link}>
<Button
{...passedProps}
navigate={route || link}
@ -413,7 +413,7 @@ function SideNavigation(props: Props) {
const { hideForUnauth, ...passedProps } = linkProps;
return !email && hideForUnauth && IS_WEB ? null : (
<li key={linkProps.icon} className="mobile-only">
<li key={linkProps.title} className="mobile-only">
<Button
{...passedProps}
navigate={linkProps.link}