Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

feat(Loader): add component#685

Merged
layershifter merged 14 commits intomasterfrom
feat/loader
Jan 11, 2019
Merged

feat(Loader): add component#685
layershifter merged 14 commits intomasterfrom
feat/loader

Conversation

@layershifter
Copy link
Member

@layershifter layershifter commented Jan 4, 2019

Fixes #666.

Initial specification: stardust-ui/specifications#5


Loader vs Spinner, Indicator...

See spec.

Missing color prop

Will be added after #647.

label vs content/children

SUIR calls this text, Fabric calls this label. Also, this part is a slot, so it cannot be directly children.
Opened there for feedback.

@layershifter
Copy link
Member Author

@jurokapsiar Juraj, please take a look on this PR. It seems that we need to add there some accessibility attributes.

@layershifter layershifter changed the title [WIP] feat(Loader): add component feat(Loader): add component Jan 8, 2019
@layershifter
Copy link
Member Author

  • labelPosition was added to spec
  • indicator slot is now configurable via prop

Thanks to @mnajdova for suggestions.

Copy link
Member

@levithomason levithomason left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@layershifter layershifter merged commit b6ddbe0 into master Jan 11, 2019
@layershifter layershifter deleted the feat/loader branch January 11, 2019 09:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants