-
Notifications
You must be signed in to change notification settings - Fork 178
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
fix(app): alignment in thermocycler card cycle info #5320
Conversation
Codecov Report
@@ Coverage Diff @@
## release_3.17.0 #5320 +/- ##
===============================================
Coverage 63.12% 63.12%
===============================================
Files 1061 1061
Lines 30161 30161
===============================================
Hits 19038 19038
Misses 11123 11123
Continue to review full report at Codecov.
|
e43967d
to
6fd1eaf
Compare
6fd1eaf
to
70f2cc5
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.
Tested - working as intended
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
This PR fixes an alignment issue within the Thermocycler's run card cycle info.
The Time remaining should be on a new row beneath the cycle and step counts, but they were all scrunching together.
This also repairs a regression in formatting the time remaining, that was introduced in the shift away from moment in #5127 . For instance, ten seconds remaining was displaying as 19:00:10 and it should correctly display as 00:00:10.