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(app): scroll to current split from jump to tick, qol tweaks to run preview #12218

Merged
merged 4 commits into from
Mar 3, 2023

Conversation

b-cooper
Copy link
Contributor

@b-cooper b-cooper commented Mar 2, 2023

Overview

Momentarily highlight jumped-to commands with a new color, maintain the current step highlight without a fade, various small design qa comments addressed.

Closes RLAB-307, RLAB-308

Changelog

  • change highlight color for tick clicks to warning enabled
  • change current step highlight to never decay and track with protocol progress
  • fix off by one error that placed ticks in slightly wrong location compared to inner progress meter percentage
  • remove '?/?' and instead show nothing if analysis and run not present

Risk assessment

low

…to run preview

Momentarily highlight jumped-to commands with a  new color, maintain the current step highlight
without a fade, various small design qa comments addressed.

Closes RLAB-307, RLAB-308
@b-cooper b-cooper requested a review from a team as a code owner March 2, 2023 20:18
@b-cooper b-cooper requested review from brenthagen and a team and removed request for a team and brenthagen March 2, 2023 20:18
@codecov
Copy link

codecov bot commented Mar 2, 2023

Codecov Report

Merging #12218 (0d5ddcb) into edge (089fa48) will increase coverage by 0.04%.
The diff coverage is 16.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##             edge   #12218      +/-   ##
==========================================
+ Coverage   74.00%   74.04%   +0.04%     
==========================================
  Files        2195     2191       -4     
  Lines       60364    60393      +29     
  Branches     6207     6260      +53     
==========================================
+ Hits        44672    44718      +46     
+ Misses      14278    14235      -43     
- Partials     1414     1440      +26     
Flag Coverage Δ
app 72.52% <16.66%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
app/src/organisms/RunPreview/index.tsx 13.63% <0.00%> (+0.59%) ⬆️
app/src/organisms/RunProgressMeter/Tick.tsx 10.52% <0.00%> (ø)
app/src/organisms/RunProgressMeter/index.tsx 39.13% <0.00%> (ø)
app/src/pages/Devices/ProtocolRunDetails/index.tsx 89.09% <60.00%> (-1.30%) ⬇️
...s/ProtocolRun/SetupLabware/CurrentOffsetsModal.tsx 79.54% <0.00%> (-10.46%) ⬇️
...ganisms/ChooseRobotToRunProtocolSlideout/index.tsx 67.50% <0.00%> (-6.03%) ⬇️
.../organisms/LabwarePositionCheck/ResultsSummary.tsx 80.00% <0.00%> (-5.72%) ⬇️
...colRun/SetupLabware/LaunchLabwarePositionCheck.tsx 61.29% <0.00%> (-2.05%) ⬇️
...ecules/PythonLabwareOffsetSnippet/createSnippet.ts 85.91% <0.00%> (-1.59%) ⬇️
app/src/organisms/ChooseProtocolSlideout/index.tsx 63.93% <0.00%> (-1.19%) ⬇️
... and 8 more

Copy link
Contributor

@SyntaxColoring SyntaxColoring left a comment

Choose a reason for hiding this comment

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

Reviewed in-person.

@b-cooper b-cooper merged commit 0b9f6fc into edge Mar 3, 2023
@b-cooper b-cooper deleted the app_run-current-step-highlight branch March 3, 2023 19:50
y3rsh added a commit that referenced this pull request Mar 6, 2023
* edge:
  refactor(protocol-engine): Shorten error docstrings (#12230)
  chore(app): update electron-builder version from v24 alpha to v24 (#12233)
  chore(app): add new icons for ODD hi-fi (#12231)
  docs: more release docs (#12226)
  fix(app, odd): check isOnDevice true instead of if it is null (#12228)
  docs: remove references to make bump in releasing (#12222)
  refactor(app): scroll to current split from jump to tick, qol tweaks to run preview (#12218)
  fix(app): rewire cli and jupyter snippets to offsets modals  (#12180)
  feat(app): handle non-deterministic protocols (#12173)
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.

2 participants