-
Notifications
You must be signed in to change notification settings - Fork 8
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
Menu and Popover don't close on button click #7
Labels
Comments
Good point! I will update this as soon as possible. But probably not before tomorrow. |
Reminder for myself: Adding a trailing slash to a website URL breaks the link. (Probably the static web app configuration) |
DavidVollmers
added a commit
that referenced
this issue
Aug 31, 2023
* toggle menu, listbox and popover when clicking the button instead of just opening it * add tests for #7
This is fixed in version 0.8.3. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Neither the HeadlessUI Menu nor Popover components are closing when their respective buttons are clicked, as tested on the examples on https://ignis.dvolper.dev/docs/components/headlessUI/. I get the same behavior on both Chrome 116.0.5845.111 and Firefox 117.0.
I think it is because the
Click()
functions are callingOpen()
only, instead of checkingMenu
state and toggling?PopoverButton.Click()
:Ignis/packages/Tailwind/Ignis.Components.HeadlessUI/PopoverButton.cs
Line 104 in 589fe64
MenuButton.Click()
:Ignis/packages/Tailwind/Ignis.Components.HeadlessUI/MenuButton.cs
Line 111 in 589fe64
The text was updated successfully, but these errors were encountered: