vmware-archive / clarity

Clarity is a scalable, accessible, customizable, open source design system built with web components. Works with any JavaScript framework, built for enterprises, and designed to be inclusive.
http://clarity.design
MIT License
6.43k stars 763 forks source link

feat(angular): do not preserve width of loading buttons #6563

Closed steve-haar closed 2 years ago

steve-haar commented 2 years ago

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

What is the current behavior?

Currently, loading buttons preserve their width when they switched to the loading state.

fixes https://github.com/vmware/clarity/issues/5967

What is the new behavior?

Loading buttons will no longer preserve their width. This also fixes styling issues when a button was initialized in the loading state.

Does this PR introduce a breaking change?

Other Information

The previous fix for same size icons is no longer needed if the button is not going to preserve its width.

vmwclabot commented 2 years ago

@steve-haar, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <john.doe@email.org> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

netlify[bot] commented 2 years ago

✔️ Deploy Preview for zzz-angular-clarity-design-old ready!

🔨 Explore the source changes: 9f04487b697d5a5925a493605e4e7fdc4e1c0669

🔍 Inspect the deploy log: https://app.netlify.com/sites/zzz-angular-clarity-design-old/deploys/61dc9624408f9f000774d8ce

😎 Browse the preview: https://deploy-preview-6563--zzz-angular-clarity-design-old.netlify.app

steve-haar commented 2 years ago

closing in favor of https://github.com/vmware/clarity/pull/6573