.clang-format

@@ -4,9 +4,11 @@ ColumnLimit:     100
 
 TabWidth:                2
 ContinuationIndentWidth: 4
-MaxEmptyLinesToKeep    : 2
+MaxEmptyLinesToKeep:     2
 
 AllowShortBlocksOnASingleLine:              false
 AllowShortIfStatementsOnASingleLine:        false
 AllowShortLoopsOnASingleLine:               false
 AllowShortFunctionsOnASingleLine:           Empty
+JavaScriptWrapImports:                      true
+

.github/branch-manager.yml

@@ -1,7 +1,7 @@
 enabled: true
 
 branches:
-  - branchName: 7.2.x
+  - branchName: 7.3.x
     label: "target: patch"
   - branchName: 7.x
     label: "target: minor"

.huskyrc.json

@@ -0,0 +1,5 @@
+{
+  "hooks": {
+    "pre-commit": "check-clang-format \"\\`yarn format\\`\""
+  }
+}

CHANGELOG.md

@@ -1,3 +1,19 @@
+## 7.3.2 "jute-box" (2019-02-11)
+
+
+### Bug Fixes
+
+* **autocomplete:** set aria-haspopup ([#15079](https://github.com/angular/material2/issues/15079)) ([45bd5d4](https://github.com/angular/material2/commit/45bd5d4))
+* **checkbox:** unable to click to select if text is marked ([#15062](https://github.com/angular/material2/issues/15062)) ([4819c5b](https://github.com/angular/material2/commit/4819c5b)), closes [#14967](https://github.com/angular/material2/issues/14967)
+* **drag-drop:** set class when item or list is disabled ([#14769](https://github.com/angular/material2/issues/14769)) ([c543cf6](https://github.com/angular/material2/commit/c543cf6)), closes [#14760](https://github.com/angular/material2/issues/14760)
+* **drag-drop:** standalone draggable drag class not being applied with OnPush change detection ([#14727](https://github.com/angular/material2/issues/14727)) ([2f63195](https://github.com/angular/material2/commit/2f63195))
+* **icon:** add notranslate class ([#14889](https://github.com/angular/material2/issues/14889)) ([39fa928](https://github.com/angular/material2/commit/39fa928))
+* **menu:** prevent removal of mat-elevation class ([#15035](https://github.com/angular/material2/issues/15035)) ([8069641](https://github.com/angular/material2/commit/8069641))
+* **slide-toggle:** input element should use switch role ([#15073](https://github.com/angular/material2/issues/15073)) ([3af7748](https://github.com/angular/material2/commit/3af7748)), closes [#14949](https://github.com/angular/material2/issues/14949)
+* **tree:** opposite direction padding not being reset on change ([#14816](https://github.com/angular/material2/issues/14816)) ([ae4a5ae](https://github.com/angular/material2/commit/ae4a5ae))
+
+
+
 ## 7.3.1 "liquidcrystal-sandwich" (2019-02-04)
 
 

e2e/BUILD.bazel

@@ -4,27 +4,43 @@ load("@angular//:index.bzl", "protractor_web_test_suite")
 load("//tools:defaults.bzl", "ts_library")
 
 ts_library(
-  name = "e2e_specs_lib",
-  srcs = glob(["**/*.ts"]),
+  name = "e2e_utils_lib",
+  srcs = glob(["util/**/*.ts"]),
   tsconfig = ":tsconfig.json",
   deps = [
     "@matdeps//@types/jasmine",
     "@matdeps//protractor"
   ]
 )
 
-protractor_web_test_suite(
-  name = "e2e",
+# For each spec file in the "components/" folder, we declare a Bazel target
+# that builds the JavaScript outputs for the e2e spec file.
+[ts_library(
+   name = "%s_specs_lib" % spec_file,
+   srcs = [spec_file],
+   tsconfig = ":tsconfig.json",
+   deps = [
+     "@matdeps//@types/jasmine",
+     "@matdeps//protractor",
+     ":e2e_utils_lib",
+   ]
+ ) for spec_file in glob(["components/**/*.spec.ts"])]
+
+# For each spec file in the "components/" folder, we declare a Protractor web test
+# suite target that runs the given e2e spec against the e2e-app devserver.
+[protractor_web_test_suite(
+  name = "%s_e2e" % spec_file,
   configuration = ":protractor.conf.js",
   on_prepare = ":start-devserver.js",
   server = "//src/e2e-app:devserver",
   deps = [
     "@matdeps//protractor",
-    ":e2e_specs_lib"
+    ":%s_specs_lib" % spec_file,
+    ":e2e_utils_lib",
   ],
   data = [
     "@angular//packages/bazel/src/protractor/utils",
     "//tools/axe-protractor",
   ],
-)
+) for spec_file in glob(["components/**/*.spec.ts"])]
 

guides/theming-your-components.md

@@ -94,6 +94,6 @@ For example:
   outline-color: mat-color($accent, lighter);
 
   // Get a constrast color for a hue by adding `-contrast` to any other key.
-  border-color: mat-color($primary, '100-constrast');
+  border-color: mat-color($primary, '100-contrast');
 }
 ```

package.json

@@ -24,9 +24,10 @@
     "stage-release": "ts-node --project tools/release/ tools/release/stage-release.ts",
     "publish-release": "ts-node --project tools/release/ tools/release/publish-release.ts",
     "check-release-output": "ts-node --project tools/release tools/release/check-release-output.ts",
-    "preinstall": "node ./tools/npm/check-npm.js"
+    "preinstall": "node ./tools/npm/check-npm.js",
+    "format": "git-clang-format"
   },
-  "version": "7.3.1",
+  "version": "7.3.2",
   "requiredAngularVersion": ">=7.0.0",
   "dependencies": {
     "@angular/animations": "^7.2.1",
@@ -79,6 +80,7 @@
     "axe-webdriverjs": "^1.1.1",
     "browser-sync": "^2.26.3",
     "chalk": "^1.1.3",
+    "clang-format": "^1.2.4",
     "codelyzer": "^4.5.0",
     "conventional-changelog": "^3.0.5",
     "dgeni": "^0.4.10",
@@ -98,6 +100,7 @@
     "hammerjs": "^2.0.8",
     "highlight.js": "^9.11.0",
     "http-rewrite-middleware": "^0.1.6",
+    "husky": "^1.3.1",
     "inquirer": "^6.2.0",
     "jasmine-core": "^3.3.0",
     "karma": "^3.1.4",

src/cdk/drag-drop/directives/drag.spec.ts

@@ -461,6 +461,25 @@ describe('CdkDrag', () => {
       expect(element.classList).not.toContain('cdk-drag-dragging');
     }));
 
+    it('should add a class while an element is being dragged with OnPush change detection',
+      fakeAsync(() => {
+        const fixture = createComponent(StandaloneDraggableWithOnPush);
+        fixture.detectChanges();
+
+        const element = fixture.componentInstance.dragElement.nativeElement;
+
+        expect(element.classList).not.toContain('cdk-drag-dragging');
+
+        startDraggingViaMouse(fixture, element);
+
+        expect(element.classList).toContain('cdk-drag-dragging');
+
+        dispatchMouseEvent(document, 'mouseup');
+        fixture.detectChanges();
+
+        expect(element.classList).not.toContain('cdk-drag-dragging');
+      }));
+
     it('should not add a class if item was not dragged more than the threshold', fakeAsync(() => {
       const fixture = createComponent(StandaloneDraggable, [], 5);
       fixture.detectChanges();
@@ -536,8 +555,12 @@ describe('CdkDrag', () => {
       fixture.detectChanges();
       const dragElement = fixture.componentInstance.dragElement.nativeElement;
 
+      expect(dragElement.classList).not.toContain('cdk-drag-disabled');
+
       fixture.componentInstance.dragInstance.disabled = true;
+      fixture.detectChanges();
 
+      expect(dragElement.classList).toContain('cdk-drag-disabled');
       expect(dragElement.style.transform).toBeFalsy();
       dragElementViaMouse(fixture, dragElement, 50, 100);
       expect(dragElement.style.transform).toBeFalsy();
@@ -2115,9 +2138,14 @@ describe('CdkDrag', () => {
       const fixture = createComponent(DraggableInDropZone);
       fixture.detectChanges();
       const dragItems = fixture.componentInstance.dragItems;
+      const dropElement = fixture.componentInstance.dropInstance.element.nativeElement;
+
+      expect(dropElement.classList).not.toContain('cdk-drop-list-disabled');
 
       fixture.componentInstance.dropInstance.disabled = true;
+      fixture.detectChanges();
 
+      expect(dropElement.classList).toContain('cdk-drop-list-disabled');
       expect(dragItems.map(drag => drag.element.nativeElement.textContent!.trim()))
           .toEqual(['Zero', 'One', 'Two', 'Three']);
 
@@ -2821,6 +2849,17 @@ class StandaloneDraggable {
   boundarySelector: string;
 }
 
+@Component({
+  changeDetection: ChangeDetectionStrategy.OnPush,
+  template: `
+    <div cdkDrag #dragElement style="width: 100px; height: 100px; background: red;"></div>
+  `
+})
+class StandaloneDraggableWithOnPush {
+  @ViewChild('dragElement') dragElement: ElementRef<HTMLElement>;
+  @ViewChild(CdkDrag) dragInstance: CdkDrag;
+}
+
 @Component({
   template: `
     <svg><g

src/cdk/drag-drop/directives/drag.ts

@@ -28,6 +28,7 @@ import {
   ViewContainerRef,
   OnChanges,
   SimpleChanges,
+  ChangeDetectorRef,
 } from '@angular/core';
 import {coerceBooleanProperty} from '@angular/cdk/coercion';
 import {Observable, Observer, Subject, merge} from 'rxjs';
@@ -69,6 +70,7 @@ export function CDK_DRAG_CONFIG_FACTORY(): DragRefConfig {
   exportAs: 'cdkDrag',
   host: {
     'class': 'cdk-drag',
+    '[class.cdk-drag-disabled]': 'disabled',
     '[class.cdk-drag-dragging]': '_dragRef.isDragging()',
   },
   providers: [{provide: CDK_DRAG_PARENT, useExisting: CdkDrag}]
@@ -174,11 +176,12 @@ export class CdkDrag<T = any> implements AfterViewInit, OnChanges, OnDestroy {
     @Optional() private _dir: Directionality,
 
     /**
-     * @deprecated `viewportRuler` and `dragDropRegistry` parameters
+     * @deprecated `viewportRuler`, `dragDropRegistry` and `_changeDetectorRef` parameters
      * to be removed. Also `dragDrop` parameter to be made required.
      * @breaking-change 8.0.0.
      */
-    dragDrop?: DragDrop) {
+    dragDrop?: DragDrop,
+    private _changeDetectorRef?: ChangeDetectorRef) {
 
 
     // @breaking-change 8.0.0 Remove null check once the paramter is made required.
@@ -191,7 +194,7 @@ export class CdkDrag<T = any> implements AfterViewInit, OnChanges, OnDestroy {
 
     this._dragRef.data = this;
     this._syncInputs(this._dragRef);
-    this._proxyEvents(this._dragRef);
+    this._handleEvents(this._dragRef);
   }
 
   /**
@@ -312,13 +315,17 @@ export class CdkDrag<T = any> implements AfterViewInit, OnChanges, OnDestroy {
     });
   }
 
-  /**
-   * Proxies the events from a DragRef to events that
-   * match the interfaces of the CdkDrag outputs.
-   */
-  private _proxyEvents(ref: DragRef<CdkDrag<T>>) {
+  /** Handles the events from the underlying `DragRef`. */
+  private _handleEvents(ref: DragRef<CdkDrag<T>>) {
     ref.started.subscribe(() => {
       this.started.emit({source: this});
+
+      // Since all of these events run outside of change detection,
+      // we need to ensure that everything is marked correctly.
+      if (this._changeDetectorRef) {
+        // @breaking-change 8.0.0 Remove null check for _changeDetectorRef
+        this._changeDetectorRef.markForCheck();
+      }
     });
 
     ref.released.subscribe(() => {
@@ -327,6 +334,13 @@ export class CdkDrag<T = any> implements AfterViewInit, OnChanges, OnDestroy {
 
     ref.ended.subscribe(() => {
       this.ended.emit({source: this});
+
+      // Since all of these events run outside of change detection,
+      // we need to ensure that everything is marked correctly.
+      if (this._changeDetectorRef) {
+        // @breaking-change 8.0.0 Remove null check for _changeDetectorRef
+        this._changeDetectorRef.markForCheck();
+      }
     });
 
     ref.entered.subscribe(event => {

src/cdk/drag-drop/directives/drop-list.ts

@@ -62,6 +62,7 @@ export interface CdkDropListInternal extends CdkDropList {}
   host: {
     'class': 'cdk-drop-list',
     '[id]': 'id',
+    '[class.cdk-drop-list-disabled]': 'disabled',
     '[class.cdk-drop-list-dragging]': '_dropListRef.isDragging()',
     '[class.cdk-drop-list-receiving]': '_dropListRef.isReceiving()',
   }

src/cdk/drag-drop/drag-drop.md

@@ -70,12 +70,14 @@ by the directives:
 
 | Selector            | Description                                                              |
 |---------------------|--------------------------------------------------------------------------|
-| `.cdk-drop-list`    | Corresponds to the `cdkDropList` container.                                  |
+| `.cdk-drop-list`    | Corresponds to the `cdkDropList` container.                              |
 | `.cdk-drag`         | Corresponds to a `cdkDrag` instance.                                     |
+| `.cdk-drag-disabled`| Class that is added to a disabled `cdkDrag`.                             |
 | `.cdk-drag-preview` | This is the element that will be rendered next to the user's cursor as they're dragging an item in a sortable list. By default the element looks exactly like the element that is being dragged. |
 | `.cdk-drag-placeholder` | This is element that will be shown instead of the real element as it's being dragged inside a `cdkDropList`. By default this will look exactly like the element that is being sorted. |
 | `.cdk-drop-list-dragging` | A class that is added to `cdkDropList` while the user is dragging an item.  |
-| `.cdk-drop-list-receiving` | A class that is added to `cdkDropList` when it can receive an item that is being dragged inside a connected drop list.  |
+| `.cdk-drop-list-disabled` | A class that is added to `cdkDropList` when it is disabled.  |
+| `.cdk-drop-list-receiving`| A class that is added to `cdkDropList` when it can receive an item that is being dragged inside a connected drop list.  |
 
 ### Animations
 The drag-and-drop module supports animations both while sorting an element inside a list, as well as

src/cdk/tree/BUILD.bazel

@@ -26,6 +26,7 @@ ng_test_library(
     "@rxjs",
     "@rxjs//operators",
     "//src/cdk/collections",
+    "//src/cdk/bidi",
     ":tree",
   ],
 )

src/cdk/tree/padding.ts

@@ -67,8 +67,8 @@ export class CdkTreeNodePadding<T> implements OnDestroy {
               private _element: ElementRef<HTMLElement>,
               @Optional() private _dir: Directionality) {
     this._setPadding();
-    if (this._dir) {
-      this._dir.change.pipe(takeUntil(this._destroyed)).subscribe(() => this._setPadding());
+    if (_dir) {
+      _dir.change.pipe(takeUntil(this._destroyed)).subscribe(() => this._setPadding());
     }
   }
 
@@ -87,9 +87,12 @@ export class CdkTreeNodePadding<T> implements OnDestroy {
   }
 
   _setPadding() {
+    const element = this._element.nativeElement;
     const padding = this._paddingIndent();
     const paddingProp = this._dir && this._dir.value === 'rtl' ? 'paddingRight' : 'paddingLeft';
+    const resetProp = paddingProp === 'paddingLeft' ? 'paddingRight' : 'paddingLeft';
 
-    this._renderer.setStyle(this._element.nativeElement, paddingProp, padding);
+    this._renderer.setStyle(element, paddingProp, padding);
+    this._renderer.setStyle(element, resetProp, '');
   }
 }