Merge branch 'master' into nrobakova/refactor-keyboardNav-tests

Author: kdinev

CHANGELOG.md

@@ -6,13 +6,34 @@ All notable changes for each version of this project will be documented in this
 
 ### General
 - Added support for the Ivy renderer.
-- **Breaking Changes** The following classes have been renamed. Using `ng update` will apply automatically migrate your project to use the new names.
+- **Breaking Changes** The following classes and enumarators have been renamed. Using `ng update` will apply automatically migrate your project to use the new names.
     - `IgxDropDownBase` -> `IgxDropDownBaseDirective`
     - `IgxDropDownItemBase` -> `IgxDropDownItemBaseDirective`
     - `IgxGridBaseComponent` -> `IgxGridBaseDirective`
     - `IgxRowComponent` -> `IgxRowDirective`
     - `IgxHierarchicalGridBaseComponent` -> `IgxHierarchicalGridBaseDirective`
     - `IgxMonthPickerBase` -> `IgxMonthPickerBaseDirective`
+    - `AvatarType` -> `IgxAvatarType`
+    - `Size` -> `IgxAvatarSize`
+    - `Type` -> `IgxBadgeType`
+    - `SliderType` -> `IgxSliderType`
+    - `TabsType` -> `IgxTabsType`
+
+- **Breaking Changes** Due to a breaking change in Angular 9 with Ivy, Hammer providers are no longer included by default. You can find more information at: https://github.com/angular/angular/blob/master/CHANGELOG.md#breaking-changes-9 . Because of this change the following components require `HammerModule` to be imported in the root module of the application in order for user interactions to work as expected:
+    - `IgxSlider`
+
+    The following components require `HammerModule` to be imported in the root module of the application so that their **touch** interactions work as expected:
+    - `igxGrid`
+    - `igxHierarchicalGrid`
+    - `igxTreeGrid`
+    - `igxList`
+    - `igxNavigationDrawer`
+    - `igxTimePicker`
+    - `igxMonthPicker`
+    - `igxSlider`
+    - `igxCalendar`
+    - `igxDatePicker`
+    - `igxCarousel`
 
 - `IgxGrid`, `IgxTreeGrid`, `IgxHierarchicalGrid`
     - **Breaking Change** - Hierarchical grid children no longer use the same `IgxTransactionService` instance and transaction handling should be modified to address each grid's transactions separately.
@@ -25,6 +46,21 @@ All notable changes for each version of this project will be documented in this
 - `IgxCarousel`:
     - **Breaking Changes** -The carousel slides are no longer array, they are changed to QueryList.
     - **Behavioural change** - When slides are more than 5, a label is shown instead of the indicators. The count limit of visible indicators can be changed with the input `maximumIndicatorsCount`
+- `IgxAvatar`:
+    - **Breaking Changes** - renamed the `default` enumeration member to `custom` in `IgxAvatarType`;
+- `IgxBadge`:
+    - **Breaking Changes** - renamed the `default` enumeration member to `primary` in `IgxBadgeType`;
+- `IgxCard`:
+    - **Breaking Changes** - renamed the `default` enumeration member to `elevated` in `IgxCardType`;
+    - **Breaking Changes** - renamed the `default` enumeration member to `start` in `IgxCardActionsLayout`;
+- `IgxDivider`:
+    - **Breaking Changes** - renamed the `default` enumeration member to `solid` in `IgxDividerType`;
+    - **Breaking Changes** - renamed the `isDefault` getter to `isSolid`;
+- `IgxProgress`:
+    - **Breaking Changes** - renamed the `danger` enumeration member to `error` in `IgxProgresType`;
+    - **Breaking Changes** - renamed the `danger` getter to `error`;
+- `IgxTabs`:
+    - **Breaking Changes** - The `tabsType` input property has been renamed to `type`. It should get renamed in your components via `ng update`;
 - `igxOverlay`:
     - **Behavioural Change** - `igxOverlay` - no longer persists element scrolling `out of the box`. In order to persist an element scroll position after attaching the element to an overlay, handle the exposed `onAppended` overlay event and manage/restore the scroll position.
 

package-lock.json

@@ -44,33 +44,33 @@
   },
   "private": true,
   "dependencies": {
-    "@angular/animations": "^9.0.0-rc.14",
-    "@angular/common": "^9.0.0-rc.14",
-    "@angular/compiler": "^9.0.0-rc.14",
-    "@angular/core": "^9.0.0-rc.14",
-    "@angular/forms": "^9.0.0-rc.14",
-    "@angular/platform-browser": "^9.0.0-rc.14",
-    "@angular/platform-browser-dynamic": "^9.0.0-rc.14",
-    "@angular/router": "^9.0.0-rc.14",
+    "@angular/animations": "^9.0.0",
+    "@angular/common": "^9.0.0",
+    "@angular/compiler": "^9.0.0",
+    "@angular/core": "^9.0.0",
+    "@angular/forms": "^9.0.0",
+    "@angular/platform-browser": "^9.0.0",
+    "@angular/platform-browser-dynamic": "^9.0.0",
+    "@angular/router": "^9.0.0",
     "@types/hammerjs": "^2.0.36",
     "@types/source-map": "0.5.2",
     "classlist.js": "^1.1.20150312",
     "core-js": "^2.6.2",
     "hammerjs": "^2.0.8",
     "jszip": "^3.1.5",
     "resize-observer-polyfill": "^1.5.1",
-    "rxjs": "^6.5.3",
+    "rxjs": "^6.5.4",
     "tslib": "^1.10.0",
     "web-animations-js": "^2.3.2",
     "zone.js": "~0.10.2"
   },
   "devDependencies": {
-    "@angular-devkit/build-angular": "~0.900.0-rc.12",
-    "@angular-devkit/build-ng-packagr": "~0.900.0-rc.12",
-    "@angular-devkit/schematics": "^9.0.0-rc.12",
-    "@angular/cli": "~9.0.0-rc.12",
-    "@angular/compiler-cli": "^9.0.0-rc.14",
-    "@angular/language-service": "^9.0.0-rc.14",
+    "@angular-devkit/build-angular": "~0.900.1",
+    "@angular-devkit/build-ng-packagr": "~0.900.1",
+    "@angular-devkit/schematics": "^9.0.1",
+    "@angular/cli": "~9.0.1",
+    "@angular/compiler-cli": "^9.0.0",
+    "@angular/language-service": "^9.0.0",
     "@angularclass/hmr": "^2.1.3",
     "@types/jasmine": "~3.3.5",
     "@types/jasminewd2": "~2.0.6",

package.json

@@ -28,6 +28,26 @@
         {
             "name": "IgxBaseExporter",
             "replaceWith": "IgxBaseExporterDirective"
+        },
+        {
+            "name": "AvatarType",
+            "replaceWith": "IgxAvatarType"
+        },
+        {
+            "name": "Size",
+            "replaceWith": "IgxAvatarSize"
+        },
+        {
+            "name": "Type",
+            "replaceWith": "IgxBadgeType"
+        },
+        {
+            "name": "SliderType",
+            "replaceWith": "IgxSliderType"
+        },
+        {
+            "name": "TabsType",
+            "replaceWith": "IgxTabsType"
         }
     ]
 }

projects/igniteui-angular/migrations/update-9_0_0/changes/classes.json

@@ -8,6 +8,14 @@
                 "selector": "igx-slider",
                 "type": "component"
             }
+        },
+        {
+            "name": "tabsType",
+            "replaceWith": "type",
+            "owner": {
+                "selector": "igx-tabs",
+                "type": "component"
+            }
         }
     ]
 }

projects/igniteui-angular/migrations/update-9_0_0/changes/inputs.json

@@ -43,4 +43,32 @@ describe('Update 9.0.0', () => {
             `);
         done();
     });
+
+    it('should update Enum names.', done => {
+        appTree.create(
+            '/testSrc/appPrefix/component/enum.component.ts',
+            `import { AvatarType, Size, Type, SliderType } from 'igniteui-angular';
+            `);
+
+        const tree = schematicRunner.runSchematic('migration-13', {}, appTree);
+        expect(tree.readContent('/testSrc/appPrefix/component/enum.component.ts'))
+            .toEqual(
+            `import { IgxAvatarType, IgxAvatarSize, IgxBadgeType, IgxSliderType } from 'igniteui-angular';
+            `);
+        done();
+    });
+
+    it('should update input prop from tabsType to type', done => {
+        appTree.create(
+            '/testSrc/appPrefix/component/tabs.component.html',
+            '<igx-tabs tabsType="fixed"></igx-tabs>'
+        );
+
+        const tree = schematicRunner.runSchematic('migration-13', {}, appTree);
+
+        expect(tree.readContent('/testSrc/appPrefix/component/tabs.component.html'))
+            .toEqual('<igx-tabs type="fixed"></igx-tabs>');
+
+        done();
+    });
 });

projects/igniteui-angular/migrations/update-9_0_0/index.spec.ts

@@ -72,14 +72,14 @@
         "resize-observer-polyfill": "^1.5.1"
     },
     "peerDependencies": {
-        "@angular/common": "^8.0.0",
-        "@angular/core": "^8.0.0",
-        "@angular/animations": "^8.0.0",
-        "@angular/forms": "^8.0.0",
+        "@angular/common": "^9.0.0",
+        "@angular/core": "^9.0.0",
+        "@angular/animations": "^9.0.0",
+        "@angular/forms": "^9.0.0",
         "web-animations-js": "^2.3.2"
     },
     "devDependencies": {
-        "@igniteui/angular-schematics": "^9.0.500-rc.1"
+        "@igniteui/angular-schematics": "~9.0.500-rc.3"
     },
     "ng-update": {
         "migrations": "./migrations/migration-collection.json"

projects/igniteui-angular/package.json

@@ -5,7 +5,7 @@ import {
 } from '@angular/core/testing';
 import { By } from '@angular/platform-browser';
 import { IgxIconModule } from '../icon/index';
-import { IgxAvatarComponent, AvatarType, Size } from './avatar.component';
+import { IgxAvatarComponent, IgxAvatarType, IgxAvatarSize } from './avatar.component';
 
 import { configureTestSuite } from '../test-utils/configure-suite';
 
@@ -14,7 +14,6 @@ describe('Avatar', () => {
     const baseClass = 'igx-avatar';
 
     const classes = {
-        default: `${baseClass}--default`,
         round: `${baseClass}--rounded`,
         small: `${baseClass}--small`,
         medium: `${baseClass}--medium`,
@@ -76,22 +75,22 @@ describe('Avatar', () => {
         const instance = fixture.componentInstance.avatar;
         const hostEl = fixture.debugElement.query(By.css(baseClass)).nativeElement;
 
-        expect(instance.size).toEqual(Size.SMALL);
+        expect(instance.size).toEqual(IgxAvatarSize.SMALL);
         expect(hostEl.classList).toContain(classes.small);
 
-        instance.size = Size.MEDIUM;
+        instance.size = IgxAvatarSize.MEDIUM;
         fixture.detectChanges();
-        expect(instance.size).toEqual(Size.MEDIUM);
+        expect(instance.size).toEqual(IgxAvatarSize.MEDIUM);
         expect(hostEl.classList).not.toContain(classes.medium);
 
-        instance.size = Size.LARGE;
+        instance.size = IgxAvatarSize.LARGE;
         fixture.detectChanges();
-        expect(instance.size).toEqual(Size.LARGE);
+        expect(instance.size).toEqual(IgxAvatarSize.LARGE);
         expect(hostEl.classList).not.toContain(classes.large);
 
         instance.size = 'nonsense';
         fixture.detectChanges();
-        expect(instance.size).toEqual(Size.SMALL);
+        expect(instance.size).toEqual(IgxAvatarSize.SMALL);
         expect(hostEl.classList).toContain(classes.small);
     });
 
@@ -102,13 +101,11 @@ describe('Avatar', () => {
         const instance = fixture.componentInstance.avatar;
         const hostEl = fixture.debugElement.query(By.css(baseClass)).nativeElement;
 
-        expect(instance.type).toEqual(AvatarType.DEFAULT);
         expect(instance.initials).toBeUndefined();
         expect(instance.src).toBeUndefined();
         expect(instance.icon).toBeUndefined();
 
         expect(hostEl.textContent).toEqual('TEST');
-        expect(hostEl.classList).toContain(classes.default);
     });
 
 
@@ -119,7 +116,7 @@ describe('Avatar', () => {
         const instance = fixture.componentInstance.avatar;
         const hostEl = fixture.debugElement.query(By.css(baseClass)).nativeElement;
 
-        expect(instance.type).toEqual(AvatarType.INITIALS);
+        expect(instance.type).toEqual(IgxAvatarType.INITIALS);
         expect(instance.initials).toEqual('ZK');
         expect(hostEl.querySelector('span').textContent).toEqual('ZK');
         expect(hostEl.classList).toContain(classes.initials);
@@ -131,7 +128,7 @@ describe('Avatar', () => {
         const instance = fixture.componentInstance.avatar;
         const hostEl = fixture.debugElement.query(By.css(baseClass)).nativeElement;
 
-        expect(instance.type).toEqual(AvatarType.ICON);
+        expect(instance.type).toEqual(IgxAvatarType.ICON);
         expect(instance.icon).toBeTruthy();
         expect(hostEl.classList).toContain(classes.icon);
     });
@@ -142,7 +139,7 @@ describe('Avatar', () => {
         const instance = fixture.componentInstance.avatar;
         const hostEl = fixture.debugElement.query(By.css(baseClass)).nativeElement;
 
-        expect(instance.type).toEqual(AvatarType.IMAGE);
+        expect(instance.type).toEqual(IgxAvatarType.IMAGE);
         expect(instance.image).toBeTruthy();
         expect(instance.image.nativeElement.style.backgroundImage).toBeDefined();