Skip to content

Arrow navigation fix#22

Draft
steviebuilds wants to merge 1 commit intomainfrom
arrow-navigation
Draft

Arrow navigation fix#22
steviebuilds wants to merge 1 commit intomainfrom
arrow-navigation

Conversation

@steviebuilds
Copy link
Copy Markdown
Owner

See issue #9.

@vercel
Copy link
Copy Markdown

vercel bot commented Aug 14, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
mui-nested-menu ✅ Ready (Inspect) Visit Preview Aug 14, 2022 at 11:43PM (UTC)

@steviebuilds steviebuilds linked an issue Aug 14, 2022 that may be closed by this pull request
@gkiely
Copy link
Copy Markdown

gkiely commented Mar 27, 2023

Hey @steviebaa, any chance this can be merged in? I am running into this issue.

@steviebuilds
Copy link
Copy Markdown
Owner Author

@gkiely I didn't get around to completing this one sorry. It needs some work before it is ready. I won't be able to get around to it anytime soon.

@steviebuilds steviebuilds added bug Something isn't working help wanted Extra attention is needed labels Mar 27, 2023
@gkiely
Copy link
Copy Markdown

gkiely commented Mar 28, 2023

No worries, thanks for the quick follow up. Remember what needed to be done to get it ready?

@Ranjeet-Naidu
Copy link
Copy Markdown

Hey @webzep, I'm facing the same issue. Would you happen to know when this will get merged?

@steviebuilds
Copy link
Copy Markdown
Owner Author

@Ranjeet-Naidu, its not on my radar unfortunately. It seems no one is actively working on it. Feel free to make a PR for it.

@eggei
Copy link
Copy Markdown

eggei commented Feb 25, 2026

I hope this fix gets added to the library but if you need a solution for this, we shared our solution with this package after having to add support for this in our project. It is not a drop-in element like NestedMenuItem, but is is still customizable as you wish. Data array takes all the MenuItemProps from MUI. Hope it helps, peace! https://www.npmjs.com/package/better-mui-menu

Usage:

const menuItems: MenuItem[] = [
  { id: 'save', label: 'Save', startIcon: Save, onClick: () => console.log('Save action') },
  { type: 'divider' },
  {
    label: 'Nested actions',
    startIcon: Cloud,
    items: [
      { label: 'Upload', onClick: () => console.log('Upload') },
      { label: 'Download', onClick: () => console.log('Download') },
    ],
  },
]

export function FileMenu() {
  const [anchorEl, setAnchorEl] = useState<null | HTMLElement>(null)

  return (
    <>
      <Button variant="contained" onClick={event => setAnchorEl(event.currentTarget)}>
        Open file menu
      </Button>
      <Menu
        anchorEl={anchorEl}
        open={Boolean(anchorEl)}
        onClose={() => setAnchorEl(null)}
        items={menuItems}
      />
    </>
  )
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Keyboard arrow open multiple menu without closing them

4 participants