Merge branch 'master' into chore/onboard-to-ibm-telemetry

Author: zvonimirfras

integration/ng14/package-lock.json

@@ -1,4 +1,5 @@
 import {
+	ChangeDetectorRef,
 	Directive,
 	EventEmitter,
 	HostBinding,
@@ -95,6 +96,8 @@ export class PopoverContainer {
 		"left" | "left-bottom" | "left-top" |
 		"right" | "right-bottom" | "right-top" = "bottom";
 
+	constructor(private changeDetectorRef: ChangeDetectorRef) {}
+
 	handleChange(open: boolean, event: Event) {
 		if (this.isOpen !== open) {
 			this.isOpenChange.emit(open);
@@ -106,5 +109,6 @@ export class PopoverContainer {
 			this.onClose.emit(event);
 		}
 		this.isOpen = open;
+		this.changeDetectorRef.markForCheck();
 	}
 }

integration/ng15/package-lock.json

@@ -6,6 +6,7 @@ import {
 } from "@angular/core";
 import { I18n } from "carbon-components-angular/i18n";
 import { TableItem } from "../table-item.class";
+import { TableRow } from "../table-row.class";
 import { Observable } from "rxjs";
 
 @Component({
@@ -17,6 +18,7 @@ import { Observable } from "rxjs";
 			[attr.aria-label]="getLabel() | i18nReplace:getSelectionLabelValue(row) | async"
 			[ariaLabel]="getLabel() | i18nReplace:getSelectionLabelValue(row) | async"
 			[checked]="selected"
+			[disabled]="disabled"
 			(change)="change.emit()">
 		</cds-radio>
 	`
@@ -35,6 +37,10 @@ export class TableRadio {
 		return this._label.value;
 	}
 
+	get disabled(): boolean {
+		return this.row ? !!(this.row as TableRow).disabled : false;
+	}
+	
 	/**
 	 * Used to populate the row selection checkbox label with a useful value if set.
 	 *

integration/ng16/package-lock.json

@@ -102,4 +102,11 @@ describe("Toggletip", () => {
 		fixture.detectChanges();
 		expect(toggletipEl.componentInstance.isOpen).toBeFalsy();
 	}));
+
+	it("should markForCheck given the changeDetectorRef is set", () => {
+		const spy = spyOn(toggletipEl.componentInstance.ref, "markForCheck");
+		buttonEl.nativeElement.click();
+		fixture.detectChanges();
+		expect(spy).toHaveBeenCalled();
+	});
 });

integration/ng17/package-lock.json

@@ -1,5 +1,7 @@
 import {
 	AfterViewInit,
+	ChangeDetectionStrategy,
+	ChangeDetectorRef,
 	Component,
 	ContentChild,
 	ElementRef,
@@ -23,6 +25,7 @@ import { ToggletipButton } from "./toggletip-button.directive";
  */
 @Component({
 	selector: "cds-toggletip, ibm-toggletip",
+	changeDetection: ChangeDetectionStrategy.OnPush,
 	template: `
 		<ng-content select="[cdsToggletipButton]"></ng-content>
 		<cds-popover-content>
@@ -42,8 +45,8 @@ export class Toggletip extends PopoverContainer implements AfterViewInit {
 
 	documentClick = this.handleFocusOut.bind(this);
 
-	constructor(private hostElement: ElementRef, private renderer: Renderer2) {
-		super();
+	constructor(private hostElement: ElementRef, private renderer: Renderer2, private ref: ChangeDetectorRef) {
+		super(ref);
 		this.highContrast = true;
 		this.dropShadow = false;
 	}

package-lock.json

@@ -51,4 +51,11 @@ describe("Definition tooltip", () => {
 		fixture.detectChanges();
 		expect(tooltipEl.componentInstance.isOpenChange.emit).toHaveBeenCalled();
 	});
+
+	it("should markForCheck given the changeDetectorRef is set", () => {
+		const spy = spyOn(tooltipEl.componentInstance.ref, "markForCheck");
+		buttonEl.triggerEventHandler("click", null);
+		fixture.detectChanges();
+		expect(spy).toHaveBeenCalled();
+	});
 });