fix: always use native click event rather than hammerjs's tap event

Closes #1113

Author: mattlewis92

package-lock.json

@@ -105,7 +105,6 @@
     "del-cli": "^2.0.0",
     "flatpickr": "^4.6.2",
     "font-awesome": "^4.7.0",
-    "hammerjs": "^2.0.8",
     "highlightjs-loader": "^0.2.3",
     "husky": "^3.0.4",
     "karma": "^4.2.0",

package.json

@@ -12,15 +12,7 @@ import {
 } from '@angular/core';
 import { DOCUMENT } from '@angular/common';
 import { Observable, Subject } from 'rxjs';
-import { requestIdleCallbackObservable } from './request-idle-callback';
-import { switchMapTo, takeUntil } from 'rxjs/operators';
-
-const clickElements = new Set<HTMLElement>();
-
-const eventName: string =
-  typeof window !== 'undefined' && typeof window['Hammer'] !== 'undefined'
-    ? 'tap'
-    : 'click';
+import { takeUntil } from 'rxjs/operators';
 
 @Directive({
   selector: '[mwlClick]'
@@ -35,53 +27,27 @@ export class ClickDirective implements OnInit, OnDestroy {
   constructor(
     private renderer: Renderer2,
     private elm: ElementRef<HTMLElement>,
-    @Inject(DOCUMENT) private document,
-    private zone: NgZone
+    @Inject(DOCUMENT) private document
   ) {}
 
   ngOnInit(): void {
     if (!this.clickListenerDisabled) {
-      this.renderer.setAttribute(
-        this.elm.nativeElement,
-        'data-calendar-clickable',
-        'true'
-      );
-      clickElements.add(this.elm.nativeElement);
-
-      // issue #942 - lazily initialise all click handlers after initial render as hammerjs is slow
-      requestIdleCallbackObservable()
-        .pipe(
-          switchMapTo(this.listen()),
-          takeUntil(this.destroy$)
-        )
+      this.listen()
+        .pipe(takeUntil(this.destroy$))
         .subscribe(event => {
-          // prevent child click events from firing on parent elements that also have click events
-          let nearestClickableParent = event.target as HTMLElement;
-          while (
-            !clickElements.has(nearestClickableParent) &&
-            nearestClickableParent !== this.document.body
-          ) {
-            nearestClickableParent = nearestClickableParent.parentElement;
-          }
-          const isThisClickableElement =
-            this.elm.nativeElement === nearestClickableParent;
-          if (isThisClickableElement) {
-            this.zone.run(() => {
-              this.click.next(event);
-            });
-          }
+          event.stopPropagation();
+          this.click.emit(event);
         });
     }
   }
 
   ngOnDestroy(): void {
     this.destroy$.next();
-    clickElements.delete(this.elm.nativeElement);
   }
 
   private listen() {
     return new Observable<MouseEvent>(observer => {
-      return this.renderer.listen(this.elm.nativeElement, eventName, event => {
+      return this.renderer.listen(this.elm.nativeElement, 'click', event => {
         observer.next(event);
       });
     });

projects/angular-calendar/src/modules/common/click.directive.ts

@@ -1,9 +1,6 @@
 import { NgModule } from '@angular/core';
 import { RouterModule } from '@angular/router';
-import {
-  BrowserModule,
-  HAMMER_GESTURE_CONFIG
-} from '@angular/platform-browser';
+import { BrowserModule } from '@angular/platform-browser';
 import { BrowserAnimationsModule } from '@angular/platform-browser/animations';
 import {
   NgbTabsetModule,
@@ -17,7 +14,6 @@ import { DemoAppComponent } from './demo-app.component';
 import { DemoComponent as DefaultDemoComponent } from './demo-modules/kitchen-sink/component';
 import { DemoModule as DefaultDemoModule } from './demo-modules/kitchen-sink/module';
 import { environment } from '../environments/environment';
-import { CustomHammerConfig } from './hammer-config';
 import { FormsModule } from '@angular/forms';
 import { ClipboardModule } from 'ngx-clipboard';
 
@@ -424,12 +420,6 @@ import { ClipboardModule } from 'ngx-clipboard';
       }
     )
   ],
-  providers: [
-    {
-      provide: HAMMER_GESTURE_CONFIG,
-      useClass: CustomHammerConfig
-    }
-  ],
   bootstrap: [DemoAppComponent]
 })
 export class DemoAppModule {}

projects/angular-calendar/src/modules/common/request-idle-callback.ts

@@ -19,5 +19,4 @@
  */
 
 import 'core-js/modules/es.object.entries';
-import 'hammerjs';
 import 'zone.js/dist/zone';