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

Disable autofill for time selection field #5875

Open
makeavish opened this issue Sep 6, 2024 · 9 comments · May be fixed by #6279
Open

Disable autofill for time selection field #5875

makeavish opened this issue Sep 6, 2024 · 9 comments · May be fixed by #6279
Assignees

Comments

@makeavish
Copy link
Member

makeavish commented Sep 6, 2024

Description: The time selection field has autocomplete enabled, which causes issues with password managers injecting values.

• Suggestion: Disable autocomplete for the time selection field to avoid interference from password managers.
• Steps to reproduce: Use a password manager while selecting time in the UI.

It'd be great if you could add autocomplete="off" to the time selection fields. My password manager keeps injecting otherwise.

@Rahul-2006
Copy link

Hi @makeavish
I want to work on this issue , can you please guide me the approach and assign it to me
As i am a beginner i want to take up this issue and help my open source journey

@dhirajudhani
Copy link

Hey @makeavish I like contribute on this issue can you assign me pleas

@abansal21
Copy link

Hi @makeavish, is this issue fixed?
I would like to take this issue and try to fix it.
I am fairly new to open source, this will serve as a kickstart for my journey.

@Chirag-S-Kotian
Copy link

i want to solve this please help me to find the exact code

@makeavish
Copy link
Member Author

@YounixM / @vikrantgupta25 can you please guide them?

@Sachin-chaurasiya
Copy link

Hello @YounixM , I would like to work on this issue, could you please assign it to me?

@YounixM
Copy link
Member

YounixM commented Oct 26, 2024

Hey@Sachin-chaurasiya : have assigned this to you.

@Sachin-chaurasiya
Copy link

Hey@Sachin-chaurasiya : have assigned this to you.

Thanks @YounixM.

@Sachin-chaurasiya
Copy link

Hello @YounixM, I have raised a PR, please review #6279

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

Successfully merging a pull request may close this issue.

7 participants