Disabled accordions should not have status - Githubissues

jpmorganchase / salt-ds

React UI components built with a focus on accessibility, customisation and ease-of-use

https://www.saltdesignsystem.com

Apache License 2.0

109 stars 78 forks source link

Disabled accordions should not have status #3549

Open joshwooding opened 2 weeks ago

joshwooding commented 2 weeks ago

Package name(s)

Core (@salt-ds/core)

Package version(s)

latest

Description

Disabled accordions should not have status, this aligns with the behaviour in Input.

Steps to reproduce

https://codesandbox.io/p/sandbox/disabled-open-accordion-538x9h

Expected behavior

No response

Operating system

[ ] macOS
[ ] Windows
[ ] Linux
[ ] iOS
[ ] Android

Browser

[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

Are you a JPMorgan Chase & Co. employee?

[ ] I am an employee of JPMorgan Chase & Co.

origami-z commented 2 weeks ago

Need to document overall approach around readonly, disable and status. Decision seems to be not clear on the design side.