fix: restore compatibility with angular 6 and 7

Fixes #97

Author: mattlewis92

package-lock.json

@@ -41,38 +41,38 @@
   },
   "homepage": "https://github.com/mattlewis92/angular-draggable-droppable#readme",
   "devDependencies": {
-    "@angular-devkit/build-angular": "~0.803.8",
-    "@angular-devkit/build-ng-packagr": "~0.803.8",
-    "@angular/cli": "^8.3.8",
-    "@angular/common": "^8.2.9",
-    "@angular/compiler": "^8.2.9",
-    "@angular/compiler-cli": "^8.2.9",
-    "@angular/core": "^8.2.9",
-    "@angular/language-service": "^8.2.9",
-    "@angular/platform-browser": "^8.2.9",
-    "@angular/platform-browser-dynamic": "^8.2.9",
+    "@angular-devkit/build-angular": "~0.13.0",
+    "@angular-devkit/build-ng-packagr": "~0.13.0",
+    "@angular/cli": "^7.3.9",
+    "@angular/common": "^7.2.15",
+    "@angular/compiler": "^7.2.15",
+    "@angular/compiler-cli": "^7.2.15",
+    "@angular/core": "^7.2.15",
+    "@angular/language-service": "^7.2.15",
+    "@angular/platform-browser": "^7.2.15",
+    "@angular/platform-browser-dynamic": "^7.2.15",
     "@compodoc/compodoc": "^1.1.10",
-    "@types/chai": "^4.2.3",
+    "@types/chai": "4.1.4",
     "@types/mocha": "^5.2.7",
     "@types/node": "^12.7.11",
     "@types/sinon": "^7.5.0",
-    "@types/sinon-chai": "^3.2.3",
+    "@types/sinon-chai": "3.2.0",
     "angular-cli-ghpages": "^0.5.3",
     "chai": "^4.2.0",
     "codecov": "^3.6.1",
-    "codelyzer": "^5.0.1",
+    "codelyzer": "^4.5.0",
     "commitizen": "^4.0.3",
     "concurrently": "^5.0.0",
     "copyfiles": "^2.1.1",
-    "core-js": "^3.2.1",
+    "core-js": "^2.6.9",
     "cz-conventional-changelog": "^3.0.2",
     "husky": "^3.0.8",
     "karma": "^4.3.0",
     "karma-chrome-launcher": "^3.1.0",
     "karma-coverage-istanbul-reporter": "^2.1.0",
     "karma-mocha": "^1.3.0",
     "mocha": "^6.2.1",
-    "ng-packagr": "^5.4.0",
+    "ng-packagr": "^4.7.1",
     "prettier": "^1.18.2",
     "pretty-quick": "^1.11.1",
     "rollup": "^1.23.1",
@@ -85,17 +85,15 @@
     "sinon-chai": "^3.3.0",
     "standard-version": "^7.0.0",
     "ts-node": "^8.4.1",
-    "tsickle": "^0.37.0",
-    "tslib": "^1.10.0",
+    "tsickle": "^0.34.0",
+    "tslib": "^1.9.0",
     "tslint": "^5.20.0",
-    "tslint-config-mwl": "^0.6.0",
-    "typescript": "~3.5.3",
+    "tslint-config-mwl": "^0.5.2",
+    "typescript": "~3.2.4",
     "validate-commit-msg": "^2.14.0",
-    "zone.js": "~0.9.1"
-  },
-  "peerDependencies": {
-    "@angular/core": ">=6.0.0 <10.0.0"
+    "zone.js": "^0.8.29"
   },
+  "peerDependencies": {},
   "config": {
     "commitizen": {
       "path": "node_modules/cz-conventional-changelog"

package.json

@@ -37,11 +37,11 @@ describe('draggable directive', () => {
     `
   })
   class TestComponent {
-    @ViewChild(DraggableDirective, { static: true })
+    @ViewChild(DraggableDirective)
     draggable: DraggableDirective;
-    @ViewChild('draggableElement', { static: true })
+    @ViewChild('draggableElement')
     draggableElement: ElementRef<HTMLDivElement>;
-    @ViewChild('ghostElementTemplateRef', { static: true })
+    @ViewChild('ghostElementTemplateRef')
     ghostElementTemplateRef: TemplateRef<any>;
     dragPointerDown = sinon.spy();
     dragStart = sinon.spy();
@@ -97,7 +97,7 @@ describe('draggable directive', () => {
     ]
   })
   class ScrollTestComponent extends TestComponent {
-    @ViewChild(DraggableScrollContainerDirective, { static: true })
+    @ViewChild(DraggableScrollContainerDirective)
     scrollContainer: DraggableScrollContainerDirective;
   }
 

projects/angular-draggable-droppable/src/lib/draggable.directive.spec.ts

@@ -53,13 +53,13 @@ describe('droppable directive', () => {
     ]
   })
   class TestComponent {
-    @ViewChild(DraggableDirective, { static: true })
+    @ViewChild(DraggableDirective)
     draggable: DraggableDirective;
-    @ViewChild(DroppableDirective, { static: true })
+    @ViewChild(DroppableDirective)
     droppable: DroppableDirective;
-    @ViewChild('droppableElement', { static: true })
+    @ViewChild('droppableElement')
     droppableElement: ElementRef<HTMLDivElement>;
-    @ViewChild('draggableElement', { static: true })
+    @ViewChild('draggableElement')
     draggableElement: ElementRef<HTMLDivElement>;
     dragEvent = sinon.spy();
     drop = sinon.spy();
@@ -123,7 +123,7 @@ describe('droppable directive', () => {
     ]
   })
   class ScrollTestComponent extends TestComponent {
-    @ViewChild(DraggableScrollContainerDirective, { static: true })
+    @ViewChild(DraggableScrollContainerDirective)
     scrollContainer: DraggableScrollContainerDirective;
   }
 

projects/angular-draggable-droppable/src/lib/droppable.directive.spec.ts

@@ -70,7 +70,7 @@ export class DroppableDirective implements OnInit, OnDestroy {
    * Called when a draggable element is dropped on this element
    */
   @Output()
-  drop = new EventEmitter<DropEvent>(); // tslint:disable-line no-output-native
+  drop = new EventEmitter<DropEvent>(); // tslint:disable-line no-output-named-after-standard-event
 
   currentDragSubscription: Subscription;
 

projects/angular-draggable-droppable/src/lib/droppable.directive.ts

@@ -1,6 +1,6 @@
 // This file is required by karma.conf.js and loads recursively all the .spec and framework files
 
-import 'core-js/es/reflect';
+import 'core-js/es7/reflect';
 import 'zone.js/dist/zone';
 import 'zone.js/dist/long-stack-trace-zone';
 import 'zone.js/dist/async-test';

projects/angular-draggable-droppable/src/test.ts

@@ -2,16 +2,15 @@
   "compileOnSave": false,
   "compilerOptions": {
     "baseUrl": "./",
-    "downlevelIteration": true,
     "importHelpers": true,
     "outDir": "./dist/out-tsc",
     "sourceMap": true,
     "declaration": false,
-    "module": "esnext",
+    "module": "es2015",
     "moduleResolution": "node",
     "emitDecoratorMetadata": true,
     "experimentalDecorators": true,
-    "target": "es2015",
+    "target": "es5",
     "typeRoots": [
       "node_modules/@types"
     ],

browserslist renamed to src/browserslist

@@ -3,8 +3,8 @@
   "rules": {
     "no-inferrable-types": false,
     "no-unused-expression": [false],
-    "component-max-inline-declarations": false,
-    "use-component-selector": false,
+    "max-inline-declarations": false,
+    "enforce-component-selector": false,
     "no-unused-css": false,
     "no-empty-interface": false,
     "jsdoc-format": false