mui / mui/material-ui

[Tabs][material-ui] Scroll buttons in RTL mode not using slots component

Open
#39,296 9 comments 0 reactions 1 assignee View on GitHub

@akshayw1 is already working on this.

Since Oct 6, 2023.

package: lab scope: tabs type: bug
Dominant language
JavaScript
Stars
99.1k
Forks
32.5k
Avg merge
2d 17h
Merged PRs (30d)
106

Description

Duplicates
  • I have searched the existing issues
Latest version
  • I have tested the latest version
Steps to reproduce 🕹

Link to live example: https://codesandbox.io/s/https-github-com-mui-material-ui-issues-39296-grwzpm

Steps:

  1. Using the codesandbox above
  2. Depending on your screen size you may need to adjust it to see the icons appear when tabs overflow
  3. Verify the custom Icon's show
  4. Update the dir const on line 44 of App.js to be rtl
  5. Icons used for start and end are not as provided
Current behavior 😯

Custom Icons displayed in ltr mode that are set in TabList slots

        slots={{
          EndScrollButtonIcon: RightArrowIcon,
          StartScrollButtonIcon: LeftArrowIcon
        }}
Screenshot 2023-10-03 at 21 29 57

When theme is set to rtl default icons are used, not the custom icons

Screenshot 2023-10-03 at 21 31 56
Expected behavior 🤔

Icons provided via slots are used in both ltr and rtl display modes

Context 🔦

No response

Your environment 🌎
npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Contributor guide

Open the contributing guide

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.