mui / material-ui

Material UI: Comprehensive React component library that implements Google's Material Design. Free forever.
https://mui.com/material-ui/
MIT License
93.43k stars 32.16k forks source link

ButtonGroup retains divider color when disabled and variant="text" #36735

Closed pricebaldwin closed 1 year ago

pricebaldwin commented 1 year ago

Duplicates

Latest version

Steps to reproduce 🕹

Link to live example: https://codesandbox.io/s/green-violet-1o2jb7?file=/demo.tsx

Steps:

  1. Create a button group like the following:
      <ButtonGroup
        color="error"
        orientation="vertical"
        aria-label="vertical contained button group"
        variant="text"
        disabled
      >
        {buttons}
      </ButtonGroup>

Current behavior 😯

When disabled, the button group retains its color in the divider lines.

image

Expected behavior 🤔

In all other variants, all colors go into the gray scale when disabled.

Context 🔦

No response

Your environment 🌎

npx @mui/envinfo ``` System: OS: macOS 13.2.1 Binaries: Node: 18.12.1 - ~/.nvm/versions/node/v18.12.1/bin/node Yarn: 1.22.19 - /opt/homebrew/bin/yarn npm: 9.1.2 - ~/.nvm/versions/node/v18.12.1/bin/npm Browsers: Chrome: 111.0.5563.146 Edge: Not Found Firefox: 111.0.1 Safari: 16.3 npmPackages: @emotion/react: 11.10.5 @emotion/styled: 11.10.5 @mui/icons-material: 5.11.0 @mui/lab: 5.0.0-alpha.118 @mui/material: 5.11.15 @mui/system: 5.11.9 @mui/utils: 5.11.9 @mui/x-date-pickers: 5.0.18 @types/react: 17.0.53 react: 17.0.2 react-dom: 17.0.2
Vishruta-Patil commented 1 year ago

Can I work on this issue?

DavidBoyer11 commented 1 year ago

A friend of mine and I are working on this for a University project because we have seen nothing has been done for a few weeks. Hope that is ok!