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

refactor(ui): update home page animations #15809

Merged
merged 10 commits into from
Nov 8, 2019
Merged

Conversation

alexpaxton
Copy link
Contributor

@alexpaxton alexpaxton commented Nov 8, 2019

  • Rebuild homepage cards using clockface Panel and CSS Grid
  • Emphasize "Load Data" panel
  • Add buttons into each panel
  • Update animations to not look disabled
  • Upgrade clockface dependency to 1.0.5 to utilize button mouse interactions

Screen Shot 2019-11-08 at 11 37 43 AM

  • CHANGELOG.md updated with a link to the PR (not the Issue)
  • Well-formatted commit messages
  • Rebased/mergeable
  • Tests pass
  • http/swagger.yml updated (if modified Go structs or API)
  • Documentation updated or issue created (provide link to issue/pr)
  • Signed CLA (if not already signed)

@alexpaxton alexpaxton requested a review from a team November 8, 2019 17:12
@ghost ghost requested review from drdelambre and hoorayimhelping and removed request for a team November 8, 2019 17:12
Copy link
Contributor

@drdelambre drdelambre left a comment

Choose a reason for hiding this comment

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

nit about state management, but nice work overall

</div>
)
const GettingStarted: FunctionComponent<WithRouterProps> = ({
params: {orgID},
Copy link
Contributor

Choose a reason for hiding this comment

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

it would be super cool if this came from the org store and not the router. feels weird, because we are including the router anyways, but by having redux be the single source of truth and only using the router as an event marker / generator, it makes the code easier to manage overall

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's a good call, I'll update this guy. We gon' catch all them loose threads

@@ -3,34 +3,31 @@
------------------------------------------------------------------------------
*/

$collector-graphic-anim-time: 0.5s;
$collector-graphic-anim-time: 0.1s;
Copy link
Contributor

Choose a reason for hiding this comment

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

SNAPPY

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Need 4 speeeeed

@alexpaxton alexpaxton merged commit b6356fc into master Nov 8, 2019
@alexpaxton alexpaxton deleted the home-page-animations branch November 8, 2019 18:08
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.

None yet

3 participants