From 8151f00bf27ba7387843e4398fc35b69da26fa24 Mon Sep 17 00:00:00 2001
From: Seth Foster <seth@opentrons.com>
Date: Thu, 5 Nov 2020 17:29:47 -0500
Subject: [PATCH 1/2] refactor(app): add more information on xy jog

In user testing, it's not entirely clear why you would want to enable xy
jog when jogging to the deck, or where you would want to jog to. This
adds more information.
---
 app/src/components/CalibrationPanels/SaveZPoint.js        | 8 ++++++--
 .../CalibrationPanels/__tests__/SaveZPoint.test.js        | 2 +-
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/app/src/components/CalibrationPanels/SaveZPoint.js b/app/src/components/CalibrationPanels/SaveZPoint.js
index a74ec8ddefe..cf26f31771a 100644
--- a/app/src/components/CalibrationPanels/SaveZPoint.js
+++ b/app/src/components/CalibrationPanels/SaveZPoint.js
@@ -64,8 +64,9 @@ const TO_USE_Z =
   'button to use this z position for the rest of deck calibration'
 const CALIBRATION_HEALTH_TO_DETERMINE =
   'button to determine how this position compares to the previously saved z-axis calibration coordinate'
-const ALLOW_HORIZONTAL_TEXT =
-  'Need to jog across the deck to align the pipette in slot 5?'
+const ALLOW_HORIZONTAL_TEXT = 'Reveal XY jog controls to move across deck'
+const ALLOW_XY_JOG_INSTRUCTIONS =
+  'If the pipette is over the embossed 5, on the ridge of the slot, or hard to see, reveal the jog controls to move the pipette across the deck.'
 
 const contentsBySessionType: {
   [SessionType]: {
@@ -180,6 +181,9 @@ export function SaveZPoint(props: CalibrationPanelProps): React.Node {
           {THEN}
           <b>{` '${buttonText}' `}</b>
           {isHealthCheck ? CALIBRATION_HEALTH_TO_DETERMINE : TO_USE_Z}.
+          <br />
+          <br />
+          {ALLOW_XY_JOG_INSTRUCTIONS}
         </Text>
         <video
           key={demoAsset}
diff --git a/app/src/components/CalibrationPanels/__tests__/SaveZPoint.test.js b/app/src/components/CalibrationPanels/__tests__/SaveZPoint.test.js
index c5c27cfc857..53f4c85f2ea 100644
--- a/app/src/components/CalibrationPanels/__tests__/SaveZPoint.test.js
+++ b/app/src/components/CalibrationPanels/__tests__/SaveZPoint.test.js
@@ -114,7 +114,7 @@ describe('SaveZPoint', () => {
     })
     wrapper
       .find(
-        'button[children="Need to jog across the deck to align the pipette in slot 5?"]'
+        'button[children="Reveal XY jog controls to move across deck"]'
       )
       .invoke('onClick')({ preventDefault: () => {} })
     jogDirections.forEach(direction => {

From 137a1d5a3d85c262f3d439517c3932aaa8adb2d8 Mon Sep 17 00:00:00 2001
From: Seth Foster <seth@opentrons.com>
Date: Thu, 5 Nov 2020 17:45:22 -0500
Subject: [PATCH 2/2] I really need to set up a git hook for this

---
 .../components/CalibrationPanels/__tests__/SaveZPoint.test.js | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/app/src/components/CalibrationPanels/__tests__/SaveZPoint.test.js b/app/src/components/CalibrationPanels/__tests__/SaveZPoint.test.js
index 53f4c85f2ea..34f145817eb 100644
--- a/app/src/components/CalibrationPanels/__tests__/SaveZPoint.test.js
+++ b/app/src/components/CalibrationPanels/__tests__/SaveZPoint.test.js
@@ -113,9 +113,7 @@ describe('SaveZPoint', () => {
       expect(getJogButton(wrapper, dir).exists()).toBe(false)
     })
     wrapper
-      .find(
-        'button[children="Reveal XY jog controls to move across deck"]'
-      )
+      .find('button[children="Reveal XY jog controls to move across deck"]')
       .invoke('onClick')({ preventDefault: () => {} })
     jogDirections.forEach(direction => {
       getJogButton(wrapper, direction).invoke('onClick')()