-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add Connection Health to VPNCard #58
Conversation
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.
R+ with comments
src/components/vpncard.js
Outdated
transform: scale(1.1); | ||
} | ||
.unstable .subline { | ||
color: orange; |
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.
#FFA436
- let's add an env var since we need this in a few places
src/components/vpncard.js
Outdated
color: orange; | ||
} | ||
.noSignal .subline { | ||
color: red; |
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.
#FF6A75
- also needs env var
svg { | ||
height: 48px; | ||
width: 48px; | ||
transition: all 3s; |
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.
should this be .3s
? 3 seconds seems like a sloooow transition.
bbbc4b6
to
2400cae
Compare
Depends on: mozilla-mobile/mozilla-vpn-client#9854
Add's a stability attribute for the
<vpn-card>
element. Swapping out the time string for that status.Screen.Recording.2024-09-09.at.15.23.43.mov