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

Delay submit in Desktop Safari #4137

Merged
merged 1 commit into from Nov 8, 2023
Merged

Delay submit in Desktop Safari #4137

merged 1 commit into from Nov 8, 2023

Conversation

AlecRust
Copy link
Contributor

@AlecRust AlecRust commented Nov 8, 2023

Wait half a second on Desktop Safari, ensure spinner is consistently shown.


supersedes #4062

To ensure spinner is shown.
@AlecRust AlecRust merged commit 689114e into main Nov 8, 2023
6 checks passed
@AlecRust AlecRust deleted the delay-safari-submit branch November 8, 2023 18:51
jaapmarcus pushed a commit that referenced this pull request Nov 8, 2023
To ensure spinner is shown.
@Steveorevo
Copy link
Contributor

Ugh. Is this issue resulting in a double submit when adding a domain using the keyboard on Safari? I'm getting a:

Error: Web domain test5.dev.pw exists

Only when using the keyboard Add Web Domain -> [type your web domain] -> press enter. Doesn't

Other, non-Safari, browsers are okay.

@Steveorevo
Copy link
Contributor

Also occurs on Safari for iOS 17.x, using the “return” virtual keyboard key; iPad.

@AlecRust
Copy link
Contributor Author

See #4206

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.

None yet

3 participants