SAP / ui5-webcomponents

UI5 Web Components - the enterprise-flavored sugar on top of native APIs! Build SAP Fiori user interfaces with the technology of your choice.
https://sap.github.io/ui5-webcomponents/
Apache License 2.0
1.5k stars 258 forks source link

[Table]: table cells not navigable by keyboard #9472

Closed ee92 closed 1 week ago

ee92 commented 1 month ago

Bug Description

Table cells are not navigable by keyboard arrow keys like the SAPUI5 table: https://ui5.sap.com/#/entity/sap.m.Table/sample/sap.m.sample.Table

Affected Component

Table

Expected Behaviour

Expect individual cells to be navigable by arrow keys

Isolated Example

https://sap.github.io/ui5-webcomponents/v1/components/Table/

Steps to Reproduce

  1. tab to table
  2. use arrow keys
  3. notice only rows are navigable, not individual cells

Log Output, Stack Trace or Screenshots

No response

Priority

Medium

UI5 Web Components Version

1.24.6

Browser

Chrome

Operating System

No response

Additional Context

No response

Organization

SuccessFactors

Declaration

TeodorTaushanov commented 1 month ago

Hi @SAP/ui5-webcomponents-topic-core, can you check this issue?

ilhan007 commented 1 month ago

Hello @ee92 this request would require significant development effort and we focus on providing bug fixes for this Table mainly. Additionally, given the fact that the new Table already supports it it would be a duplicate effort. Actually the improved accessibility is one of the key advantages of the new Table and part of the decision to take on developing has been several accessibility issues filed by SFSF and because there were difficult to implement in the v1/Table with its DOM structure, we have decided to build another one.

Several things about the new Table:

As the usage of the new Table means upgrading to 2.0 and adopting other breaking changes, what do you think, is this option feasible for you?

Learn more about 2.0:

ilhan007 commented 1 week ago

Closing due to inactivity, feel free to re-trigger the discussion and we will re-open the issue.

BR, ilhan