Skip to content

fix: Loading component no longer spins off-centre#474

Merged
frankieyan merged 4 commits intodevfrom
craigcarlyle/loading-spinner-fix
Mar 22, 2021
Merged

fix: Loading component no longer spins off-centre#474
frankieyan merged 4 commits intodevfrom
craigcarlyle/loading-spinner-fix

Conversation

@craigcarlyle
Copy link
Contributor

@craigcarlyle craigcarlyle commented Mar 18, 2021

Short description

Fixes an issue where the Loading component would spin off-centre

PR Checklist

  • Executed npm run validate and made sure no errors / warnings were shown
  • Updated all static build artifacts (npm run build-all)

Test Plan

  • Run storybook and navigate to the Loading component
  • Example should not longer be spinning in a wacky fashion

Demo

BeforeAfter
before.mov
after.mov

@craigcarlyle craigcarlyle changed the title fix: Loader no longer spins off-centre fix: Loading component no longer spins off-centre Mar 18, 2021
@craigcarlyle craigcarlyle self-assigned this Mar 18, 2021
@craigcarlyle craigcarlyle requested review from a team and gnapse and removed request for a team March 18, 2021 20:38
@craigcarlyle
Copy link
Contributor Author

cc: @scottlovegrove for visibility 👀

@frankieyan frankieyan merged commit 55c42b0 into dev Mar 22, 2021
@frankieyan frankieyan deleted the craigcarlyle/loading-spinner-fix branch March 22, 2021 07:47
@frankieyan
Copy link
Member

Released under v9.1.1 and v9.2.0-beta.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants