Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added some additional functionality #9

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

damienhoneyford
Copy link

This component is brilliant, and just saved me a lot of work so thank you!

There were a couple of things I needed for my implementation that may be of use to others so I thought I'd set up a PR in case you agree.

Added an optional callback parameter to the Options that can be used to fire custom actions when a menu item is moved to or from the overflow menu, e.g. change icon size
Added new recalculateBreakpoints method to allow the breakpoints created on initialization to be re-initialized to support dynamic menus
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant