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

[ui5-cell ui5-button]: A disabled button as last element in the tab chain will prevent focus to first element in the tab chain #8852

Closed
1 task done
f16r opened this issue Apr 25, 2024 · 1 comment · Fixed by #8957
Assignees
Labels
bug This issue is a bug in the code CBC Medium Prio TOPIC B

Comments

@f16r
Copy link

f16r commented Apr 25, 2024

Bug Description

A disabled button inside a table cell as last element in the tab chain will prevent to move focus to first element in the tab chain.

Affected Component

ui5-cell ui5-button

Expected Behaviour

Moving focus to the first element in the tab chain should be possible without marking a disabled button explicitly with tabIndex -1

Isolated Example

https://codesandbox.io/p/sandbox/ui5-webcomponents-playground-forked-q9dqxj?file=%2Findex.html

Steps to Reproduce

  1. Go to isolated example
  2. Click on last row to focus the row
  3. Click tab -> tabbing is stuck
  4. Workaround: Set tabIndex on button explicitly to -1. Tabbing is not stuck anymore (expected behaviour)

Log Output, Stack Trace or Screenshots

No response

Priority

Medium

UI5 Web Components Version

1.24.0

Browser

Chrome

Operating System

No response

Additional Context

No response

Organization

CBC

Declaration

  • I’m not disclosing any internal or sensitive information.
@f16r f16r added the bug This issue is a bug in the code label Apr 25, 2024
@NHristov-sap
Copy link
Contributor

Hello @ui5-webcomponents-topic-b,

I am forwarding this issue to you, because the issue is reproducible and the solution proposed by the reporter seems to be valid. Please check and fix.

Best Regards,
Nikolay Hristov
UI5 WebComponents Dispatcher

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug in the code CBC Medium Prio TOPIC B
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

3 participants