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

[material-ui][Autocomplete] Requiring double escape key press to trigger clearOnEscape #41059

Closed
EyaOuenniche opened this issue Feb 11, 2024 · 6 comments
Assignees
Labels
component: autocomplete This is the name of the generic UI component, not the React module! package: material-ui Specific to @mui/material

Comments

@EyaOuenniche
Copy link
Contributor

EyaOuenniche commented Feb 11, 2024

Steps to reproduce

Link to live example: https://mui.com/material-ui/react-autocomplete/

Steps:
1.Go to the Autocomplete demo page
2.Focus on the Autocomplete input field with clearOnEscape set to true
3.Select any item or enter text into the input field.
4.Press the Escape key once and observe the behavior.
5. Press the Escape key a second time to see the input clear.

Current behavior

The input is cleared only after pressing the Escape key twice.

Expected behavior

A single press of the Escape key should clear the input when clearOnEscape is enabled.

Context

No response

Your environment

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: Autocomplete, clearOnEscape,bug

@EyaOuenniche EyaOuenniche added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Feb 11, 2024
@danilo-leal danilo-leal changed the title Autocomplete Component Requires Double Press of Escape Key to Trigger clearOnEscape [material-ui][Autocomplete] Requiring double escape key press to trigger clearOnEscape Feb 12, 2024
@danilo-leal danilo-leal added package: material-ui Specific to @mui/material component: autocomplete This is the name of the generic UI component, not the React module! labels Feb 12, 2024
@DiegoAndai
Copy link
Member

Hey @EyaOuenniche, thanks for the report!

I'm not able to repro this on Chrome, Safari, or Firefox. I'm running MacOS.

Screen.Recording.2024-02-16.at.15.10.23.mov

Could you provide more information about the OS and browser you're using? Thanks.

@DiegoAndai DiegoAndai assigned DiegoAndai and unassigned mj12albert Feb 16, 2024
@DiegoAndai DiegoAndai added status: waiting for author Issue with insufficient information and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Feb 16, 2024
@EyaOuenniche
Copy link
Contributor Author

hello,
I'm running windows 11, I tried the issue on Microsoft edge.

@github-actions github-actions bot added status: waiting for maintainer These issues haven't been looked at yet by a maintainer and removed status: waiting for author Issue with insufficient information labels Feb 17, 2024
@michaldudak
Copy link
Member

I tried playing with the second autocomplete in the Playground but I couldn't see any issues (Edge 123 on Windows 11).

@DiegoAndai
Copy link
Member

Sadly, without a repro it isn't easy to debug this. @EyaOuenniche do you think there might be something in your setup or settings that might be related to this?

@DiegoAndai DiegoAndai added status: waiting for author Issue with insufficient information and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Feb 19, 2024
@EyaOuenniche
Copy link
Contributor Author

Hello,
Yes There was a bug here actually in the Playground.
image
Thankfully, it got fixed.
Thank you!

@github-actions github-actions bot added status: waiting for maintainer These issues haven't been looked at yet by a maintainer and removed status: waiting for author Issue with insufficient information labels Feb 19, 2024
@DiegoAndai
Copy link
Member

@EyaOuenniche you're welcome 😊

@DiegoAndai DiegoAndai closed this as not planned Won't fix, can't repro, duplicate, stale Feb 19, 2024
@zannager zannager removed the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: autocomplete This is the name of the generic UI component, not the React module! package: material-ui Specific to @mui/material
Projects
None yet
Development

No branches or pull requests

6 participants