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

How to set tabindex on mat-pass-toggle-visibility button #255

Closed
dmosberger opened this issue Dec 18, 2019 · 3 comments
Closed

How to set tabindex on mat-pass-toggle-visibility button #255

dmosberger opened this issue Dec 18, 2019 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@dmosberger
Copy link

Bug Report or Feature Request (mark with an x)

- [ ] bug report -> please search issues before submitting
- [x] feature request

OS and Version?

Linux Ubuntu 18.04.3

Versions

Angular CLI: 8.3.12
Node: 10.18.0
OS: linux x64
Angular: 8.2.11
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package Version

@angular-devkit/architect 0.803.12
@angular-devkit/build-angular 0.803.12
@angular-devkit/build-optimizer 0.803.12
@angular-devkit/build-webpack 0.803.12
@angular-devkit/core 8.3.12
@angular-devkit/schematics 8.3.12
@angular/cdk 8.2.3
@angular/cli 8.3.12
@angular/flex-layout 8.0.0-beta.27
@angular/material 8.2.3
@ngtools/webpack 8.3.12
@schematics/angular 8.3.12
@schematics/update 0.803.12
rxjs 6.5.3
typescript 3.5.3
webpack 4.39.2

Repro steps

There seems to be no way to control the tabindex value for the button created by mat-pass-toggle-visibility. Thus, there doesn't seem to be a way to setup the tab key so that it would go from the password field to the confirm password field with a single tab (it'll always go to the password's visibility button first).

The log given by the failure

Desired functionality

Maybe there could be an input value for mat-pass-toggle-visibility to set the desired tabindex value?

Mention any other details that might be useful

@AnthonyNahas
Copy link
Member

would you provide a PR for that ?

@msenosiain
Copy link
Contributor

here @AnthonyNahas :)

@dmosberger
Copy link
Author

dmosberger commented Jun 21, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants