Minor fixes

src/components/Beacon.tsx

@@ -10,10 +10,20 @@ export default class JoyrideBeacon extends React.Component<BeaconProps> {
   constructor(props: BeaconProps) {
     super(props);
 
-    if (!props.beaconComponent) {
-      const head = document.head || document.getElementsByTagName('head')[0];
-      const style = document.createElement('style');
-      const css = `
+    if (props.beaconComponent) {
+      return;
+    }
+
+    const head = document.head || document.getElementsByTagName('head')[0];
+    const style = document.createElement('style');
+
+    style.id = 'joyride-beacon-animation';
+
+    if (props.nonce) {
+      style.setAttribute('nonce', props.nonce);
+    }
+
+    const css = `
         @keyframes joyride-beacon-inner {
           20% {
             opacity: 0.9;
@@ -41,16 +51,9 @@ export default class JoyrideBeacon extends React.Component<BeaconProps> {
         }
       `;
 
-      style.id = 'joyride-beacon-animation';
-
-      if (props.nonce) {
-        style.setAttribute('nonce', props.nonce);
-      }
+    style.appendChild(document.createTextNode(css));
 
-      style.appendChild(document.createTextNode(css));
-
-      head.appendChild(style);
-    }
+    head.appendChild(style);
   }
 
   componentDidMount() {

src/components/Overlay.tsx

@@ -43,9 +43,9 @@ export default class JoyrideOverlay extends React.Component<OverlayProps, State>
     this.isActive = true;
 
     /* istanbul ignore else */
-    if (!disableScrolling) {
+    if (process.env.NODE_ENV !== 'production') {
       /* istanbul ignore else */
-      if (process.env.NODE_ENV === 'development' && hasCustomScrollParent(element, true)) {
+      if (!disableScrolling && hasCustomScrollParent(element, true)) {
         log({
           title: 'step has a custom scroll parent and can cause trouble with scrolling',
           data: [{ key: 'parent', value: this.scrollParent }],

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/global.d.ts

@@ -0,0 +1,8 @@
+declare global {
+  namespace NodeJS {
+    interface ProcessEnv {
+      NODE_ENV: 'development' | 'production' | 'test';
+    }
+  }
+}
+export {};

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();
     });
   });