Skip to content

Commit

Permalink
Remove TODO, moved to PR comment
Browse files Browse the repository at this point in the history
  • Loading branch information
AliceR committed Nov 21, 2024
1 parent 0d0d464 commit c263635
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions app/scripts/components/common/page-header/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,6 @@ const PageHeader: React.FC<PageHeaderProps> = ({
linkProperties,
mode
}) => {
//@TODO: toggle expanded on window resize! Or on click anywhere?
// weird, should be implemented on the library side of things, see https://vscode.dev/github/NASA-IMPACT/veda-ui/blob/1137-implement-new-ds-page-header/node_modules/%40uswds/uswds/packages/usa-header/src/index.js#L155-L156

const [expanded, setExpanded] = useState<boolean>(false);
const [isOpen, setIsOpen] = useState<boolean[]>(
mainNavItems.map(() => false)
Expand All @@ -42,7 +39,6 @@ const PageHeader: React.FC<PageHeaderProps> = ({
});
};

// @TODO: we should close the menu when the user clicks on a link (internal or other cta)
const themeMode = mode ? `mode-${mode}` : 'mode-light';

const primaryItems = useMemo(
Expand Down

0 comments on commit c263635

Please sign in to comment.