Skip to content
This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

Refactor ability to shift+TAB from table cell #2150

Open
kolaps33 opened this issue Dec 5, 2019 · 0 comments
Open

Refactor ability to shift+TAB from table cell #2150

kolaps33 opened this issue Dec 5, 2019 · 0 comments
Labels
vsts Paired with ticket in vsts

Comments

@kolaps33
Copy link
Collaborator

kolaps33 commented Dec 5, 2019

Bug Report

With the following PR
#2147

there is coming the workaround about ability to shift tab from table cell, as well from table header cell and from table content cel.

actionHandlers = {
    unsetRowTabbable: e => {
      this.rowRef.current.setAttribute('tabindex', '-1')

The same issue is observed as well for focuszone:
https://developer.microsoft.com/en-us/fabric#/controls/web/focuszone
Example:
Nesting FocusZones in list rows
Navigate to the cell and then press shift+TAB. Focus goes on the row not away from the list (in our case table).

In our table case it is even more complicated as in the table header row we use

  shouldFocusInnerElementWhenReceivedFocus: true,

what it means that without the workaround user is not able to shift+TAB at all.

@pkumarie2011 pkumarie2011 added the vsts Paired with ticket in vsts label Dec 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
vsts Paired with ticket in vsts
Projects
None yet
Development

No branches or pull requests

2 participants