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

calcite-input: input loses focus on every keypress when login info saved (safari) #5026

Open
gpbmike opened this issue Jul 26, 2022 · 13 comments
Labels
1 - assigned Issues that are assigned to a sprint and a team member. blocked This issue is blocked by another issue. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. p - low Issue is non core or affecting less that 10% of people using the library
Milestone

Comments

@gpbmike
Copy link
Contributor

gpbmike commented Jul 26, 2022

Actual Behavior

Using Safari, If you use calcite-input in login form and choose to save email/password, when you return the inputs will lose focus on every keypress.

Blocked issues: #8126

calcite-input-safari-blur.mov

Expected Behavior

User should be able to freely type email and password without having to re-focus on the field.

Reproduction Sample

https://mikehorn.dev/calcite-input-saved-login/

https://github.com/gpbmike/calcite-input-saved-login (code)

Reproduction Steps

  1. Use Safari.
  2. Enter any email/password.
  3. Save login info to browser.
  4. Return to form.
  5. Try to type email and password.
  6. Every keypress causes input to blur.

Reproduction Version

beta.86

Relevant Info

No response

Regression?

No response

@gpbmike gpbmike added 0 - new New issues that need assignment. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. needs triage Planning workflow - pending design/dev review. labels Jul 26, 2022
@github-actions
Copy link
Contributor

It looks like this issue is missing some information:

Issues without reproduction samples may not be prioritized. If you were unable to create a sample, please try to answer any questions that arise once development begins. Thanks for your understanding.

@github-actions github-actions bot added the incomplete issue report New issues missing important information, and unless provided, they will be closed after 5 days. label Jul 26, 2022
@benelan benelan removed the incomplete issue report New issues missing important information, and unless provided, they will be closed after 5 days. label Jul 27, 2022
@geospatialem geospatialem removed the needs triage Planning workflow - pending design/dev review. label Aug 3, 2022
@eriklharper eriklharper self-assigned this Aug 15, 2022
@benelan benelan added 1 - assigned Issues that are assigned to a sprint and a team member. and removed 0 - new New issues that need assignment. labels Aug 15, 2022
@jcfranco jcfranco added this to the 2023 December Priorities milestone May 26, 2023
@anveshmekala
Copy link
Contributor

When inert is added similar to PR #5221 , the native HTML5 error messages are not showing up in safari /chrome or edge.

@jcfranco
Copy link
Member

@eriklharper ☝️ this throws a wrench in the current solution. 🔧💨✋

@geospatialem geospatialem added spike Issues that need quick investigations for time estimations, prioritization, or a quick assessment. and removed research Issues that require more in-depth research or multiple team members to resolve or make decision. labels Sep 21, 2023
@eriklharper eriklharper added the 1 - assigned Issues that are assigned to a sprint and a team member. label Feb 28, 2024
@eriklharper eriklharper self-assigned this Feb 28, 2024
@eriklharper eriklharper removed the 0 - new New issues that need assignment. label Mar 5, 2024
eriklharper added a commit that referenced this issue Mar 5, 2024
…ser autofill and demonstrating that ElementInternals allows a native input in a shadowRoot to work with autofill and external forms at the same time.
@eriklharper eriklharper added blocked This issue is blocked by another issue. and removed spike Issues that need quick investigations for time estimations, prioritization, or a quick assessment. labels Mar 5, 2024
@eriklharper
Copy link
Contributor

eriklharper commented Mar 5, 2024

Did some spike work on this, and I created a branch to track the work I did.
Based on that work I'd say that when we get to adding ElementInternals support to calcite-input, this should resolve the autofill issues with Safari (and potentially other browsers too). That branch requires creating a local ssl certificate named calcite.dev (I used mkcert as outlined in the instructions here. Thanks @gpbmike for that link! ;)) and also requires updating your HOSTS file to point calcite.dev to localhost. This is to get autofill to work.

@geospatialem
Copy link
Member

Reallocated to November to determine if the element internals resolve the above via #8126.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 - assigned Issues that are assigned to a sprint and a team member. blocked This issue is blocked by another issue. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. p - low Issue is non core or affecting less that 10% of people using the library
Projects
None yet
Development

No branches or pull requests

9 participants