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

[@mantine/core] Menu - Close menu when menu target children change #2646

Merged
merged 32 commits into from Oct 6, 2022

Conversation

wes337
Copy link
Contributor

@wes337 wes337 commented Oct 6, 2022

Will close the Menu when menu target's children change.

When the menu target is unmounted:
image

When the menu target is hidden:

Resolves #2535

@wes337 wes337 changed the title [@mantine/core] Menu - Close menu on menu target children cleanup [@mantine/core] Menu - Close menu when menu target children change Oct 6, 2022
@rtivital rtivital merged commit 03c32b5 into mantinedev:master Oct 6, 2022
@rtivital
Copy link
Member

rtivital commented Oct 6, 2022

Thanks!

@rtivital
Copy link
Member

@wes337 there are several issues reported after this PR, can you please check whether they are caused by your changes? #2693

@wes337
Copy link
Contributor Author

wes337 commented Oct 13, 2022

@wes337 there are several issues reported after this PR, can you please check whether they are caused by your changes? #2693

Sure, I will look into this

wes337 added a commit to wes337/mantine that referenced this pull request Oct 13, 2022
rtivital pushed a commit that referenced this pull request Oct 18, 2022
…#2701)

* [@mantine/core] Select: only use open/close callback when value changes

* [@mantine/core] Select: do not use early return

* [@mantine/core] Menu - revert PR #2646

* [@mantine/core] Menu - remove unused imports
rtivital pushed a commit that referenced this pull request Oct 22, 2022
…n step is larger than the difference between current value and min/max (#2656)

* [@mantine/core] Select: only use open/close callback when value changes

* [@mantine/core] Select: do not use early return

* [@mantine/core] Slider: fix numeric max and negative numbers

* [@mantine/core] Slider: return min val is value is min

* [@mantine/core] Menu - revert PR #2646

* [@mantine/core] Slider: dont return value lower than min or greater than max
rtivital pushed a commit that referenced this pull request Nov 2, 2022
* [@mantine/core] Select: only use open/close callback when value changes

* [@mantine/core] Select: do not use early return

* [@mantine/core] Menu - revert PR #2646

* [Mantine/core] Slider: fix min/max values

* [@mantine/core] Slider: remove unneeded story

* [@mantine/core] Slider: revert changes to minmax story
rtivital pushed a commit that referenced this pull request Feb 3, 2023
… MultiSelect and Autocomplete dropdown was flipped and user started searching (#3439)

* [@mantine/core] Select: only use open/close callback when value changes

* [@mantine/core] Select: do not use early return

* [@mantine/core] Menu - revert PR #2646

* [@mantine/core] recalculate dropdown position on search input change

* [@mantine/core] recalculate dropdown position on search input change
rtivital pushed a commit that referenced this pull request May 30, 2023
…with long text (#4296)

* [@mantine/core] Select: only use open/close callback when value changes

* [@mantine/core] Select: do not use early return

* [@mantine/core] Menu - revert PR #2646

* [@mantine/core] Select - remove white space pre

* [@mantine/core] Select - break all for options
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.

Hide Menu with withinPortal when it target is hidden (itself or one of parents)
2 participants