fix(InViewport): Fix module not working with aot

Reverted using injection token to reference Window and using WindowRef class

BREAKING CHANGE: Must provide the whole Provide object when providing window for browser module

Author: edoparearyee

README.md

@@ -26,10 +26,14 @@ yarn add @thisissoon/angular-inviewport
 ```ts
 import { InViewportModule } from '@thisissoon/angular-inviewport';
 
+const providers = [
+  { provide: WindowRef, useValue: window }
+];
+
 @NgModule({
   imports: [
-    // pass window object to forRoot function
-    InViewportModule.forRoot(window)
+    // provide WindowRef class by using an window object
+    InViewportModule.forRoot(providers)
   ]
 })
 export class AppModule { }

index.ts

@@ -5,5 +5,4 @@ export {
   eventWindowScroll, inViewportClass, notInViewportClass
 } from './src/app/in-viewport/shared/event-data';
 export { Viewport } from './src/app/in-viewport/shared/viewport.model';
-export { WindowRef } from './src/app/in-viewport/window/window.service';
-export { WINDOW } from './src/app/in-viewport/window/window-token';
+export { WindowRef } from './src/app/in-viewport/window/window-ref.service';

src/app/app.module.ts

@@ -2,15 +2,19 @@ import { BrowserModule } from '@angular/platform-browser';
 import { NgModule } from '@angular/core';
 
 import { AppComponent } from './app.component';
-import { InViewportModule } from './in-viewport';
+import { InViewportModule, WindowRef } from './in-viewport';
+
+const providers = [
+  { provide: WindowRef, useValue: window }
+];
 
 @NgModule({
   declarations: [
     AppComponent
   ],
   imports: [
     BrowserModule,
-    InViewportModule.forRoot(window)
+    InViewportModule.forRoot(providers)
   ],
   bootstrap: [AppComponent]
 })

src/app/in-viewport/in-viewport.directive.spec.ts

@@ -1,6 +1,6 @@
 import { fakeAsync, tick } from '@angular/core/testing';
 import { ElementRef, NgZone } from '@angular/core';
-import { WindowRef } from './window/window.service';
+import { WindowRef } from './window/window-ref.service';
 import { InViewportDirective } from './in-viewport.directive';
 import { FakeDOMStandardElement } from '../testing/dom';
 import { MockNgZone } from '../testing/mock-ng-zone';

src/app/in-viewport/in-viewport.directive.ts

@@ -1,7 +1,7 @@
 import {
   Directive, ElementRef, HostBinding, EventEmitter,
   Input, Output, OnDestroy, AfterViewInit,
-  ChangeDetectorRef, NgZone, PLATFORM_ID, Inject
+  ChangeDetectorRef, NgZone, Inject
 } from '@angular/core';
 import { Observable } from 'rxjs/Observable';
 import { Subject } from 'rxjs/Subject';
@@ -11,8 +11,7 @@ import 'rxjs/add/operator/debounceTime';
 import 'rxjs/add/operator/takeUntil';
 import 'rxjs/add/observable/merge';
 
-import { WindowRef } from './window/window.service';
-import { WINDOW } from './window/window-token';
+import { WindowRef } from './window/window-ref.service';
 import { Viewport } from './shared/viewport.model';
 import * as eventData from './shared/event-data';
 
@@ -135,7 +134,7 @@ export class InViewportDirective implements AfterViewInit, OnDestroy {
    */
   constructor(
     private el: ElementRef,
-    @Inject(WINDOW) private windowRef: WindowRef,
+    private windowRef: WindowRef,
     private cdRef: ChangeDetectorRef,
     public ngZone: NgZone
   ) { }

src/app/in-viewport/in-viewport.module.ts

@@ -1,10 +1,11 @@
 import { NgModule, ModuleWithProviders, Provider } from '@angular/core';
-import { WindowRef } from './window/window.service';
-import { WINDOW } from './window/window-token';
+import { WindowRef } from './window/window-ref.service';
 
 import { InViewportDirective } from './in-viewport.directive';
 
-const defaultWindowRef = new WindowRef();
+const defaultProviders = [
+  WindowRef
+];
 
 /**
  * A simple lightweight library for Angular 2/4+ with other dependencies
@@ -25,16 +26,14 @@ export class InViewportModule {
    * services.
    *
    * @static
-   * @param {any} [windowRef=defaultWindowRef]
+   * @param {Provider[]} [providers]
    * @returns {ModuleWithProviders}
    * @memberof InViewportModule
    */
-  public static forRoot(windowRef: any = defaultWindowRef): ModuleWithProviders {
+  public static forRoot(providers: Provider[] = defaultProviders): ModuleWithProviders {
     return {
       ngModule: InViewportModule,
-      providers: [
-        { provide: WINDOW, useValue: windowRef }
-      ]
+      providers: providers
     };
   }
 }

src/app/in-viewport/window/index.ts

@@ -1,2 +1 @@
-export * from './window.service';
-export * from './window-token';
+export * from './window-ref.service';

src/app/in-viewport/window/window.service.ts renamed to src/app/in-viewport/window/window-ref.service.ts

@@ -1,10 +1,13 @@
+import { Injectable } from '@angular/core';
+
 /**
- * Class that acts a reference to window object
+ * Service that acts a reference to window object
  * in platforms where `window` does not exists
  *
  * @export
- * @class WindowRef
+ * @property WindowRef
  */
+@Injectable()
 export class WindowRef {
   closed = null;
   defaultStatus = null;
@@ -57,7 +60,7 @@ export class WindowRef {
   resizeBy() {}
   resizeTo() {}
   removeEventListener() {}
-  scroll() { }
+  scroll() {}
   scrollBy() {}
   scrollTo() {}
   setInterval() {}