Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ui: Loader amends/improvements #10181

Merged
merged 4 commits into from
May 7, 2021
Merged

ui: Loader amends/improvements #10181

merged 4 commits into from
May 7, 2021

Conversation

johncowen
Copy link
Contributor

@johncowen johncowen commented May 5, 2021

This PR improves our loaders slightly (Preview Build):

loader

Notes:

  1. Created a JS compatible template only 'glimmer' component so we can use it with or without glimmer (e.g. in our index.html file).
  2. Added a set of rose colors.
  3. Our loader could appear off center for a small amount of time, depending on load timing, so we animated the brand loader to keep it centered when the new side navigation appears.
  4. Tweak the color of Consul::Loader to use a new 'rose' color.
  5. Move everything loader related to the app/components/ folder and add docs (BrandLoader, Consul::Loader).

Copy link
Contributor

@kaxcode kaxcode left a comment

Choose a reason for hiding this comment

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

🎨 LGTM

John Cowen added 3 commits May 7, 2021 12:10
1. Create a JS compatible template only 'glimmer' component so we can
use it with or without glimmer.
2. Add a set of `rose` colors.
3. Animate the brand loader to keep it centered when the side
navigation appears.
4. Tweak the color of Consul::Loader to use a 'rose' color.
5. Move everything loader related to the `app/components/` folder and
add docs.
@johncowen johncowen force-pushed the ui/feature/loaders branch from 6b9d572 to 23030b9 Compare May 7, 2021 11:10
@vercel vercel bot temporarily deployed to Preview – consul May 7, 2021 11:11 Inactive
@johncowen johncowen added pr/no-changelog PR does not need a corresponding .changelog entry and removed pr/no-changelog PR does not need a corresponding .changelog entry labels May 7, 2021
@vercel vercel bot temporarily deployed to Preview – consul-ui-staging May 7, 2021 11:14 Inactive
@vercel vercel bot temporarily deployed to Preview – consul May 7, 2021 11:14 Inactive
@johncowen johncowen merged commit e3f9a0f into master May 7, 2021
@johncowen johncowen deleted the ui/feature/loaders branch May 7, 2021 11:23
@hc-github-team-consul-core
Copy link
Collaborator

🍒 If backport labels were added before merging, cherry-picking will start automatically.

To retroactively trigger a backport after merging, add backport labels and re-run https://circleci.com/gh/hashicorp/consul/365659.

@hc-github-team-consul-core
Copy link
Collaborator

🍒✅ Cherry pick of commit e3f9a0f onto release/1.10.x succeeded!

hc-github-team-consul-core pushed a commit that referenced this pull request May 7, 2021
* ui: Loader amends/improvements

1. Create a JS compatible template only 'glimmer' component so we can
use it with or without glimmer.
2. Add a set of `rose` colors.
3. Animate the brand loader to keep it centered when the side
navigation appears.
4. Tweak the color of Consul::Loader to use a 'rose' color.
5. Move everything loader related to the `app/components/` folder and
add docs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme/ui Anything related to the UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants