mui / mui/material-ui

[Tabs] Update to match the specification

Open
#15,324 28 comments 3 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

breaking change design: material scope: tabs
Dominant language
JavaScript
Stars
99.1k
Forks
32.5k
Avg merge
2d 17h
Merged PRs (30d)
106

Description

Hi, I have to disagree with this change.
We are using scrollable and fullWidth together just fine, it only requires adding a minWidth to the Tab button components but otherwise it works just fine.
This is a breaking change for our layout - could this be reverted / are there other solutions to keep fullWidth and scrollable available at the same time?

Originally posted by @olee in https://github.com/mui-org/material-ui/pull/13980#issuecomment-482465309

https://material.io/design/components/tabs.html

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.

Research direction

Start by reading the linked Material Design Tabs specification and the discussion about combining scrollable with fullWidth. Inspect the Tabs implementation and its existing tests to identify the behavior changed by the referenced pull request. Done means the two options can be used together without breaking existing layouts, with tests covering the supported combination.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, react
Domain
frontend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.