fix(protocol-designer): avoid StepList exception on keydown (#7442)

- No longer ignore exceptions in TC e2e test(this is the test that often fails on Mac in CI)

Author: IanLondon

protocol-designer/cypress/integration/newProtocolWithTC.spec.js

@@ -91,13 +91,6 @@ describe('Protocols with Modules', () => {
     })
 
     it('designs protocol with thermocycler', () => {
-      // Avoid failing the test on uncaught exception
-      // Happens when writing a nickname for the labware
-      Cypress.on('uncaught:exception', (err, runnable) => {
-        console.log(err.name)
-        return false
-      })
-
       cy.openDesignPage()
       // close "setting up" modal
       cy.contains('Setting up your protocol').should('exist')

protocol-designer/src/components/steplist/StepList.js

@@ -25,7 +25,7 @@ export class StepList extends React.Component<Props> {
   handleKeyDown: (e: SyntheticKeyboardEvent<>) => void = e => {
     const { reorderSelectedStep } = this.props
     const key = e.key
-    const altIsPressed = e.getModifierState('Alt')
+    const altIsPressed = e.altKey
 
     if (altIsPressed) {
       let delta = 0