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

Spinner should not use "useDensity" #3564

Open joshwooding opened 3 weeks ago

joshwooding commented 3 weeks ago

Package name(s)

Core (@salt-ds/core)

Package version(s)

latest

Description

Using "useDensity" makes it easier to have a mismatch if components are bundled in other libraries. Spinner currently uses useDensity and Javascript plus CSS which increases the probability of an issue occuring.

Steps to reproduce

No response

Expected behavior

No response

Operating system

Browser

Are you a JPMorgan Chase & Co. employee?