Skip to content

Conversation

@mejo-
Copy link
Member

@mejo- mejo- commented Sep 11, 2024

Fixes: #5695

The tabindex was introduced with commit 2c9e7a0, but I was not able to reproduce a problem with tab navigation and/or focus trap after removing it.

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits

Fixes: #5695

The `tabindex` was introduced with commit 2c9e7a0, but I was not able to
reproduce a problem with tab navigation and/or focus trap after removing
it.

Signed-off-by: Jonas <[email protected]>
@mejo- mejo- added this to the Nextcloud 31 milestone Sep 11, 2024
@mejo- mejo- self-assigned this Sep 11, 2024
@mejo-
Copy link
Member Author

mejo- commented Sep 11, 2024

/backport to stable30

@mejo-
Copy link
Member Author

mejo- commented Sep 11, 2024

/backport to stable29

@mejo-
Copy link
Member Author

mejo- commented Sep 11, 2024

/backport to stable28

@juliusknorr
Copy link
Member

I cannot remember the exact reason, but double checked the focus order and seems all fine. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Editing in code block does not work in firefox

3 participants