Skip to content

fix(ui-buttons): disabled button with href should not receive focus#2110

Closed
joyenjoyer wants to merge 1 commit intomasterfrom
INSTUI-4687-disabled-button-should-not-receive-focus
Closed

fix(ui-buttons): disabled button with href should not receive focus#2110
joyenjoyer wants to merge 1 commit intomasterfrom
INSTUI-4687-disabled-button-should-not-receive-focus

Conversation

@joyenjoyer
Copy link
Contributor

@joyenjoyer joyenjoyer commented Aug 27, 2025

INSTUI-4687

Test:

  • add disabled and href properties to Button
  • observe if it is focusable (it should NOT be), try it in different browsers
  • try it with ScreenReaders (it should be read as a disabled/dimmed link)

@github-actions
Copy link

github-actions bot commented Aug 27, 2025

PR Preview Action v1.6.2
Preview removed because the pull request was closed.
2025-09-15 08:27 UTC

@joyenjoyer joyenjoyer force-pushed the INSTUI-4687-disabled-button-should-not-receive-focus branch from 6938dca to 4af8687 Compare September 9, 2025 13:22
@joyenjoyer joyenjoyer force-pushed the INSTUI-4687-disabled-button-should-not-receive-focus branch from 4af8687 to 481ae40 Compare September 9, 2025 13:53
@joyenjoyer joyenjoyer self-assigned this Sep 9, 2025
Copy link
Contributor

@ToMESSKa ToMESSKa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joyenjoyer In Safari (Version 18.6 (20621.3.11.11.3)), Button is still focusable (it has a focus ring) when disabled with a href. Can you please check this?

@joyenjoyer joyenjoyer closed this Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments