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

Click away #671

Merged
merged 2 commits into from
Sep 14, 2024
Merged

Click away #671

merged 2 commits into from
Sep 14, 2024

Conversation

bryannaegele
Copy link
Contributor

Expose the phx click away event that was added recently.

https://hexdocs.pm/phoenix_live_view/bindings.html#click-events

@Malian
Copy link
Collaborator

Malian commented Nov 8, 2022

Thanks for the PR @bryannaegele !

I believe a prop is missing in Surface.Components.Events to fix the tests.

Finally, could you also please add a test to Surface.Components.EventsTest.

After that, we will be able to merge.

@bryannaegele
Copy link
Contributor Author

Awesome. I couldn't figure out which file to do it in. I'll fix that up soon.

@tiagoefmoraes tiagoefmoraes merged commit 2130438 into surface-ui:main Sep 14, 2024
6 checks passed
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.

3 participants