Esri / calcite-design-system

A monorepo containing the packages for Esri's Calcite Design System
https://developers.arcgis.com/calcite-design-system/
Other
285 stars 76 forks source link

Loader - design issues #4470

Open asangma opened 2 years ago

asangma commented 2 years ago

Actual Behavior

When inline

When inline and determinate

When not inline

Expected Behavior

When inline and determinate

When not inline

Reproduction Sample

https://codepen.io/mtb-alan/pen/QWQLKvq

Reproduction Steps

  1. View the sample

Reproduction Version

latest

Relevant Info

No response

Regression?

No response

Calcite Design Recommendations

See this comment for design specs.

asangma commented 2 years ago

cc @macandcheese

asangma commented 2 years ago

It looks like inline Loader has a stroke: currentColor on it.

macandcheese commented 2 years ago

We should evaluate the need for these string props at all, if this is the only bug report maybe they aren't worth maintaining? If we do keep them, we'll need design improvements for the bugs shown in that codepen.

geospatialem commented 1 year ago

Will investigate further to help prioritize after next month's release if breaking changes are needed (May release), or if we can incorporate into an earlier release.

macandcheese commented 1 year ago

@asangma want to take a stab at some of these design improvement proposals :) ?

geospatialem commented 1 year ago

@asangma want to take a stab at some of these design improvement proposals :) ?

@asangma WDYT about taking a look into some prospective design opportunities in July?

geospatialem commented 1 year ago

Updated the issue context above to include the design considerations of inline with text per this Community thread.

jacqueskeet commented 1 month ago

Please see the Figma file for the proposed designs here:

https://www.figma.com/design/3ZUBn0JzuDTqQ1LYYtDdXT/Loader?m=dev&focus-id=1786-768

CleanShot 2024-08-16 at 15 10 55@2x

github-actions[bot] commented 1 month ago

cc @geospatialem, @brittneytewks