Skip to content

Commit

Permalink
Fixed React Hook useState is called conditionally error
Browse files Browse the repository at this point in the history
  • Loading branch information
bp-kyasuda committed Jun 23, 2023
1 parent f90574e commit 8238abd
Showing 1 changed file with 23 additions and 27 deletions.
50 changes: 23 additions & 27 deletions frontend/src/app/components/Sidebar/SidebarNav.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -56,10 +56,6 @@ const SidebarNav = ({
hideParentScrollbar,
onPageChange,
}: Props): ReactElement | null => {
if (appPages.length < 2) {
return null
}

const [expanded, setExpanded] = useState(true)
const { pageLinkBaseUrl } = React.useContext(AppContext)
const navItemsRef = useRef<HTMLUListElement>(null)
Expand Down Expand Up @@ -113,29 +109,29 @@ const SidebarNav = ({
<li key={pageName}>
<StyledSidebarNavLinkContainer>
{page.majorClass ? (
<StyledSidebarNavMajorClass>
{pageName.replace(/_/g, " ")}
</StyledSidebarNavMajorClass>
) : (
<StyledSidebarNavLink
isActive={isActive}
href={pageUrl}
onClick={e => {
e.preventDefault()
onPageChange(page.pageScriptHash as string)
if (reactDeviceDetect.isMobile) {
collapseSidebar()
}
}}
>
{page.icon && page.icon.length && (
<EmojiIcon size="lg">{page.icon}</EmojiIcon>
)}
<StyledSidebarLinkText isActive={isActive}>
{tooltipContent}
</StyledSidebarLinkText>
</StyledSidebarNavLink>
)}
<StyledSidebarNavMajorClass>
{pageName.replace(/_/g, " ")}
</StyledSidebarNavMajorClass>
) : (
<StyledSidebarNavLink
isActive={isActive}
href={pageUrl}
onClick={e => {
e.preventDefault()
onPageChange(page.pageScriptHash as string)
if (reactDeviceDetect.isMobile) {
collapseSidebar()
}
}}
>
{page.icon && page.icon.length && (
<EmojiIcon size="lg">{page.icon}</EmojiIcon>
)}
<StyledSidebarLinkText isActive={isActive}>
{tooltipContent}
</StyledSidebarLinkText>
</StyledSidebarNavLink>
)}
</StyledSidebarNavLinkContainer>
</li>
)
Expand Down

0 comments on commit 8238abd

Please sign in to comment.