microsoft / microsoft/fluentui

[Bug]: fluent-tabs keyboard navigation is not flipped in RTL

Open
#28,935 5 comments 0 reactions 1 assignee View on GitHub

@chrisdholt is already working on this.

Since Aug 21, 2023.

Fluent UI WC (v3) Resolution: Soft Close web-components
Dominant language
TypeScript
Stars
20.3k
Forks
2.9k
Avg merge
2d 9h
Merged PRs (30d)
46

Description

Library

Web Components (@fluentui/web-components)

System Info
@fluentui/web-components@3.0.0-alpha.29
Are you reporting Accessibility issue?

None

Reproduction

https://codesandbox.io/s/fuiwc-tabs-rtl-ncdrgd?file=/index.html

Bug Description
<fluent-tabs dir="rtl">
  <fluent-tab>First Tab</fluent-tab>
  <fluent-tab id="${secondTabId}">Second Tab</fluent-tab>
  <fluent-tab>Third Tab</fluent-tab>
  <fluent-tab>Fourth Tab</fluent-tab>
  <fluent-tab-panel>First Panel</fluent-tab-panel>
  <fluent-tab-panel>Second Panel</fluent-tab-panel>
  <fluent-tab-panel>Third Panel</fluent-tab-panel>
  <fluent-tab-panel>Fourth Panel</fluent-tab-panel>
</fluent-tabs>

Focus "Second Tab", press right arrow

Actual Behavior

focus goes to the left ("Third tab")

Expected Behavior

focus should go to right ("First tab")

Logs

No response

Requested priority

Normal

Products/sites affected

No response

Are you willing to submit a PR to fix?

no

Validations
  • Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • The provided reproduction is a minimal reproducible example of the bug.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.