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

[Feature Request] should have props to set tabindex on clearable icon #11202

Closed
mr-mad-e opened this issue Apr 23, 2020 · 8 comments
Closed

[Feature Request] should have props to set tabindex on clearable icon #11202

mr-mad-e opened this issue Apr 23, 2020 · 8 comments
Assignees
Labels
C: VTextField VTextField T: bug Functionality that does not work as intended/expected
Milestone

Comments

@mr-mad-e
Copy link

Problem to solve

You can't skip tabindex on clearable icon

Proposed solution

should have props to set tabbindex

:clearable-tabindex="-1"

@ghost ghost added the S: triage label Apr 23, 2020
@mr-mad-e

This comment has been minimized.

@xmtoken
Copy link

xmtoken commented May 1, 2020

I have temporarily avoided it by using directive.
https://codepen.io/xmid1001/pen/MWaEppz

@bob-lee

This comment has been minimized.

@RacerDelux
Copy link

I think it should be valid to enable it to be tabbed, but the default should have it set to tabindex=-1.
Maybe another option exposed?

@dmcknight26
Copy link

dmcknight26 commented Aug 25, 2020

Agreed, I hope this becomes optional behavior. On heavy data entry applications with advanced/power users the extra [TAB]s involved will really slow them down.

There is absolutely a use case for this though in more public facing applications where accessibility is important.

@johnleider johnleider added T: feature A new feature and removed S: triage labels Dec 1, 2020
@johnleider
Copy link
Member

I almost consider this a bug but am leaving it tagged as a Feature for now even though I'd like to get it done as part of v2 maintenance.

@johnleider johnleider added this to the v3.0.0 milestone Jul 14, 2021
@johnleider johnleider added this to To do in Vuetify 3 - Titan via automation Jul 14, 2021
@johnleider johnleider modified the milestones: v3.0.0, v2.5.x Aug 4, 2021
@Tomdanizer
Copy link

Tomdanizer commented Oct 5, 2021

I have temporarily avoided it by using directive. https://codepen.io/xmid1001/pen/MWaEppz

This workaround is semi-broken as of 2.5.2 and on for text-fields that are not pre-populated with data. 2.5.2 changed it so the clear icon is not created until there is data in the field. See https://codepen.io/tomdanizer/pen/XWgLqdg

It still works if the fields are pre-populated with data.

@johnleider johnleider modified the milestones: v2.5.x, v3.0.0 Oct 5, 2021
@KaelWD
Copy link
Member

KaelWD commented Jul 6, 2022

https://ux.stackexchange.com/q/125755/148323
https://www.scottohara.me/blog/2022/02/19/custom-clear-buttons.html

In v3 they currently aren't focusable, but also have aria-hidden which isn't great.

@KaelWD KaelWD added T: bug Functionality that does not work as intended/expected and removed T: feature A new feature labels Aug 10, 2022
@KaelWD KaelWD modified the milestones: v3.0.0, v2.6.x Aug 10, 2022
@KaelWD KaelWD closed this as completed in f8ee680 Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: VTextField VTextField T: bug Functionality that does not work as intended/expected
Projects
No open projects
Development

No branches or pull requests

9 participants