Skip to content
This repository was archived by the owner on Jan 23, 2025. It is now read-only.

ui-components: Spinner component#132

Merged
koorosh merged 1 commit intocockroachdb:masterfrom
koorosh:ui-spinner
Nov 9, 2020
Merged

ui-components: Spinner component#132
koorosh merged 1 commit intocockroachdb:masterfrom
koorosh:ui-spinner

Conversation

@koorosh
Copy link
Copy Markdown
Contributor

@koorosh koorosh commented Nov 2, 2020

ui-components // Spinner

Spinner component is animated loading indicator which can be
used in tables or on pages and indicate intermediate state.

This component doesn't reuse existing Spinner icon due to
technical limitations. Loading icon is rendered with <path>
element which is difficult to animate and make it rotate around
SVG central point. That's why current component implements its
own loading icon based on <circle> element.

Checklist

  • I have written or updated test for the changes I made
  • I have updated the README of the package I'm working in to reflect my changes
  • I have added or updated Storybook if appropriate for my changes

output

For comparison, loading indicator which is used in Admin UI right now:
spinner

Spinner component is animated loading indicator which can be
used in tables or on pages and indicate intermediate state.

This component doesn't reuse existing Spinner icon due to
technical limitations. Loading icon is rendered with <path>
element which is difficult to animate and make it rotate around
SVG central point. That's why current component implements its
own loading icon based on <circle> element.
Copy link
Copy Markdown
Contributor

@nathanstilwell nathanstilwell left a comment

Choose a reason for hiding this comment

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

This looks great. Thank you @koorosh

@koorosh koorosh merged commit dfdfb47 into cockroachdb:master Nov 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants