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

fix!: prevent Safari AutoFill to avoid click issues #7619

Merged
merged 3 commits into from
Aug 8, 2024

Conversation

vursen
Copy link
Contributor

@vursen vursen commented Aug 7, 2024

Description

The PR adds the "search" prefix to the date and time picker's id attribute. This is a trick to prevent Safari from enabling AutoFill, which is causing click issues. See the referenced ticket for more details.

Warning

@web-padawan: could be considered behavior altering fix due to the fact of changing ID attributes which can be unexpected by some users in terms of testing.

Fixes #6817

Type of change

  • Bugfix

@vursen vursen changed the title fix: prevent Safari from enabling AutoFill by using search prefix fix: prevent Safari AutoFill by adding search prefix to id attr Aug 7, 2024
@vursen vursen changed the title fix: prevent Safari AutoFill by adding search prefix to id attr fix: prevent Safari AutoFill by adding search word to id attr Aug 7, 2024
@vursen vursen changed the title fix: prevent Safari AutoFill by adding search word to id attr fix: prevent Safari AutoFill to avoid click issues Aug 7, 2024
@vursen vursen marked this pull request as ready for review August 7, 2024 13:08
@vursen vursen requested a review from web-padawan August 7, 2024 13:08
@vursen vursen changed the title fix: prevent Safari AutoFill to avoid click issues fix!: prevent Safari AutoFill to avoid click issues Aug 7, 2024
Copy link

sonarcloud bot commented Aug 7, 2024

@web-padawan web-padawan merged commit f10c4de into main Aug 8, 2024
9 checks passed
@web-padawan web-padawan deleted the fix/disable-safari-autofill branch August 8, 2024 08:42
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 24.5.0.alpha10 and is also targeting the upcoming stable 24.5.0 version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[date-time-picker] Safari's autofill prevents the pickers from opening on the first click
3 participants