Skip to content

fix: Enter key should submit forms in Safari #4815

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

Merged
merged 1 commit into from
Jun 29, 2022
Merged

Conversation

driskull
Copy link
Member

Related Issue: #4793

Summary

fix: Enter key should submit forms in Safari (#4793)

@driskull driskull requested a review from a team as a code owner June 28, 2022 17:38
@github-actions github-actions bot added the bug Bug reports for broken functionality. Issues should include a reproduction of the bug. label Jun 28, 2022
Copy link
Contributor

@eriklharper eriklharper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting the kind of hacks needed for Safari! #SafariIsTheNewIE

@driskull
Copy link
Member Author

Yep. I thought we checked and that requestSubmit worked in all browsers but I guess not :\

@driskull driskull merged commit ae432dd into master Jun 29, 2022
@driskull driskull deleted the dris0000/safari-submit branch June 29, 2022 21:18
@github-actions github-actions bot added this to the Sprint 06/27 - 07/08 milestone Jun 29, 2022
@jcfranco
Copy link
Member

@driskull @eriklharper @geospatialem do either of you recall if the original issue only had inputs inside a form? It's no longer available. The polyfill we use for Safari was applied in calcite-button and could lead to the reported error if not used in a form. If this is the case, I've submitted #6623 to address this.

@driskull
Copy link
Member Author

I don't recall. I think it just had buttons for submitting.

@eriklharper
Copy link
Contributor

I don't recall either...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug reports for broken functionality. Issues should include a reproduction of the bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants