primereact icon indicating copy to clipboard operation
primereact copied to clipboard

Password: (accessiblility) cannot access toggle button using keyboard

Open arnodemer opened this issue 1 year ago • 6 comments

Describe the bug

focus cannot be moved to Password's toggle button, then cannot toggle using keyboard

Reproducer

No response

PrimeReact version

10.5.1

React version

17.x

Language

TypeScript

Build / Runtime

Create React App (CRA)

Browser(s)

No response

Steps to reproduce the behavior

  1. go to Password toggle show case : https://primereact.org/password/#togglemask
  2. use tab to set focus on password field
  3. hit tab several times, focus is never set on toggle password button

Expected behavior

Toggle button should be able to be activated using keyboard Focus should also stay on toggle button when password state is toggled.

arnodemer avatar Feb 27 '24 10:02 arnodemer

Agreed it looks like PrimeVue has not implemented this yet either.

melloware avatar Feb 27 '24 16:02 melloware

@melloware Can I work on this?

akshayaqburst avatar Feb 28 '24 06:02 akshayaqburst

Hi ! it would be nice if we could also set a tooltip for this toggle button (password component already have one, but it's for input ). Thanks a lot !

arnodemer avatar Feb 28 '24 08:02 arnodemer

@akshayaqburst sure!

melloware avatar Feb 28 '24 09:02 melloware

@melloware Created PR for review - https://github.com/primefaces/primereact/pull/6052. Thanks,

akshayaqburst avatar Feb 28 '24 13:02 akshayaqburst

Some code review comments. Looking good so far!

melloware avatar Feb 28 '24 13:02 melloware