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

[Textbox] - onClick handler is still being passed when the component is disabled #6671

Open
1 task done
sianford opened this issue Apr 11, 2024 · 2 comments
Open
1 task done

Comments

@sianford
Copy link
Contributor

sianford commented Apr 11, 2024

Description

When the component is disabled, NVDA screen reader registers that the input field is clickable with a mouse/trackpad.

Reproduction

https://carbon.sage.com/?path=/story/textbox--disabled

Steps to reproduce

  1. Open the Textbox Disabled story in Windows.
  2. Open NVDA with the Speech Viewer switched on.
  3. Click on the content of the disabled Textbox field.
  4. Inspect the Speech Viewer window.

Expected result:
There is no response when the field is clicked on. This is not accessible at all using the keyboard (which is expected behaviour).

Actual result:
The Speech Viewer states the field is "clickable".

JIRA ticket numbers (Sage only)

SKPU-11421

Suggested solution

Remove the onClick handler for disabled inputs.

Carbon version

132.1.0

Design tokens version

No response

Relevant browsers

Firefox, Chrome, Microsoft Edge

Relevant OSs

Windows

Additional context

VoiceOver used with Safari on Mac also states "To enter text in this feild, type." indicating it also detects the field is accessible.

Confidentiality

  • I confirm there is no confidential or commercially sensitive information included.
@sianford sianford added Bug triage Triage Required labels Apr 11, 2024
@edleeks87
Copy link
Contributor

We are still passing a handler to the underlying input, we should add a ternary like we have in Textbox to solve this

@edleeks87
Copy link
Contributor

FE-6430

@edleeks87 edleeks87 added On Backlog and removed triage Triage Required labels Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants