-
Notifications
You must be signed in to change notification settings - Fork 30
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
Rui/score bar #410
Rui/score bar #410
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Running Lighthouse audit... |
Needs more discuss regarding severity ranges CC:@peterslany |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
caf06f8
to
a01a084
Compare
a01a084
to
07cdd47
Compare
https://www.loom.com/share/fbbe0f2cea764a8a9fe7760b205a04d4 This is the latest approach |
07cdd47
to
ced556b
Compare
ced556b
to
fdbb6c9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! I appreciate how you managed to solve the issue with the ranges very well 👍 I brought up some minor comments that are not blocking merging :)
src/component-library/CollateralScore/CollateralScore.style.tsx
Outdated
Show resolved
Hide resolved
src/component-library/CollateralScore/CollateralScore.style.tsx
Outdated
Show resolved
Hide resolved
896f39a
to
9225f80
Compare
9225f80
to
3ba9b10
Compare
Demo