Minor fixes

src/components/index.tsx

@@ -236,10 +236,9 @@ class Joyride extends React.Component<Props, State> {
     const { index, lifecycle } = this.state;
     const { steps } = this.props;
     const step = steps[index];
-    const intKey = window.Event ? event.which : event.keyCode;
 
     if (lifecycle === LIFECYCLE.TOOLTIP) {
-      if (intKey === 27 && step && !step.disableCloseOnEsc) {
+      if (event.code === 'Escape' && step && !step.disableCloseOnEsc) {
         this.store.close();
       }
     }
@@ -272,7 +271,7 @@ class Joyride extends React.Component<Props, State> {
     } = this.props;
     const step = getMergedStep(steps[index], this.props);
 
-    const target = getElement(step.target) as HTMLElement;
+    const target = getElement(step.target);
     const shouldScrollToStep = shouldScroll({
       isFirstStep: index === 0,
       lifecycle,

src/modules/dom.ts

@@ -196,7 +196,7 @@ export function getElementPosition(
 /**
  * Get the scrollTop position
  */
-export function getScrollTo(element: HTMLElement, offset: number, skipFix: boolean): number {
+export function getScrollTo(element: HTMLElement | null, offset: number, skipFix: boolean): number {
   if (!element) {
     return 0;
   }

src/modules/helpers.ts

@@ -25,7 +25,7 @@ interface ShouldScrollOptions {
   previousLifecycle: Lifecycle;
   scrollToFirstStep: boolean;
   step: Step;
-  target: HTMLElement;
+  target: HTMLElement | null;
 }
 
 export const isReact16 = createPortal !== undefined;

src/modules/scope.ts

@@ -128,11 +128,11 @@ export default class Scope {
       return;
     }
 
-    const target = this.element.querySelector(selector) as HTMLElement;
+    const target = this.element.querySelector(selector);
 
     /* istanbul ignore else */
     if (target) {
-      window.requestAnimationFrame(() => this.checkFocus(target));
+      window.requestAnimationFrame(() => this.checkFocus(target as HTMLElement));
     }
   };
 }

test/modules/scope.spec.tsx

@@ -51,7 +51,6 @@ describe('modules/scope', () => {
     });
 
     it('should not have created an instance', () => {
-      // expect(Scope).toHaveBeenCalledTimes(1);
       expect(scope).toBeUndefined();
     });
   });