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

report: animated chevrons #5137

Merged
merged 4 commits into from
May 18, 2018
Merged

report: animated chevrons #5137

merged 4 commits into from
May 18, 2018

Conversation

paulirish
Copy link
Member

@paulirish paulirish commented May 6, 2018

Thx to @hwikyounglee for putting up with me as I was like "OH HECK NO".
Because I enjoyed making this work. :D

seagulls2

This is dependent on #5136


https://lighthouse-chevrons.now.sh/

Copy link
Collaborator

@patrickhulce patrickhulce left a comment

Choose a reason for hiding this comment

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

wow 😮shut up and take my 💰💰

my hesitation with approving is setting the expectation we can do fancy stuff like this left and right 😆

}
.lh-chevron__line {
stroke: hsl(216, 5%, 39%);
transform-origin: 50px 50px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

can any of this be set to a variable to scale the chevron or is it all very precarious?

Copy link
Member Author

Choose a reason for hiding this comment

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

good call. done.

@hwikyounglee
Copy link

Thank you @paulirish 😊🙃🕺💃

Copy link

@hwikyounglee hwikyounglee left a comment

Choose a reason for hiding this comment

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

LGTM

@paulirish
Copy link
Member Author

my hesitation with approving is setting the expectation we can do fancy stuff like this left and right 😆

you can try this out on https://sweltering-ground.surge.sh/cnn.html
when its not zoomed in, i think the effect is pretty subtle.

regarless, in general, i think that with hwi's guidance we won't overdo things. :)

@patrickhulce
Copy link
Collaborator

oh not meant as a comment on the awesomeness of the effect all! More that I hope Hwi won't be disappointed with me 🙏 when I'm unable to do the next awesome animation she comes up with in the same amount of time @paulirish did 😆

@paulirish
Copy link
Member Author

oooooooooooh gotcha. :D

Copy link
Member

@brendankenny brendankenny left a comment

Choose a reason for hiding this comment

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

I expect only fancy stuff in our report from now on

@paulirish paulirish changed the base branch from oppoAsAnAudit to metricsdisplay-oldrebase May 7, 2018 19:20
@paulirish paulirish changed the base branch from metricsdisplay-oldrebase to metricsdisplay-oldrebase2 May 7, 2018 19:21
@brendankenny
Copy link
Member

@patrickhulce had a question for you and then we can land :)

@googlebot
Copy link

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this state. It's up to you to confirm consent of the commit author(s) and merge this pull request when appropriate.

@googlebot
Copy link

CLAs look good, thanks!

@paulirish paulirish changed the base branch from oppoAsAnAudit to master May 17, 2018 23:33
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.

6 participants