Merge branch 'master' into issue.2857

Author: Akshat55

package-lock.json

@@ -143,7 +143,7 @@
     "@carbon/icons": "11.14.0",
     "@carbon/utils-position": "1.1.4",
     "@floating-ui/dom": "1.6.3",
-    "@ibm/telemetry-js": "^1.2.1",
+    "@ibm/telemetry-js": "^1.5.0",
     "flatpickr": "4.6.13",
     "tslib": "2.3.0"
   }

package.json

@@ -16,4 +16,4 @@ gulp buildMeta
 # generate ALL the documentation
 mkdir dist/docs
 npm run build-storybook
-npm run docs:build && mv documentation dist/docs/
+npm run docs:build && mv documentation dist/docs/ && mv telemetry.yml dist/telemetry.yml

scripts/build.sh

@@ -352,7 +352,7 @@ export class DropdownList implements AbstractDropdownView, AfterViewInit, OnDest
 			}
 		}
 
-		return elemList[this.index].nativeElement;
+		return elemList[this.index]?.nativeElement;
 	}
 
 	/**
@@ -490,6 +490,7 @@ export class DropdownList implements AbstractDropdownView, AfterViewInit, OnDest
 	updateIndex() {
 		// initialize index on the first selected item or
 		// on the next non disabled item if no items are selected
+		// in case, if all items are disabled, the index value will remain same
 		const selected = this.getSelected();
 		if (selected.length) {
 			this.index = this.displayItems.indexOf(selected[0]);
@@ -513,7 +514,7 @@ export class DropdownList implements AbstractDropdownView, AfterViewInit, OnDest
 			event.preventDefault();
 			if (event.key === "ArrowDown") {
 				if (this.hasNextElement()) {
-					this.getNextElement().scrollIntoView({ block: "end" });
+					this.getNextElement()?.scrollIntoView({ block: "end" });
 				} else {
 					this.blurIntent.emit("bottom");
 				}

src/dropdown/list/dropdown-list.component.ts

@@ -8,8 +8,8 @@
 		"@carbon/utils-position",
 		"@carbon/icon-helpers",
 		"@carbon/icons",
-		"@carbon/telemetry",
-    "@floating-ui/dom",
-    "flatpickr"
+		"@floating-ui/dom",
+		"@ibm/telemetry-js",
+		"flatpickr"
 	]
 }

src/ng-package.json

@@ -8,7 +8,7 @@
     "url": "[email protected]:carbon-design-system/carbon-components-angular.git"
   },
   "scripts": {
-    "postinstall": "carbon-telemetry collect --install"
+    "postinstall": "ibmtelemetry --config=telemetry.yml"
   },
   "license": "Apache-2.0",
   "author": "IBM",
@@ -18,9 +18,9 @@
   "dependencies": {
     "@carbon/icon-helpers": "10.37.0",
     "@carbon/icons": "11.14.0",
-    "@carbon/telemetry": "0.1.0",
     "@carbon/utils-position": "1.1.4",
     "@floating-ui/dom": "1.6.3",
+    "@ibm/telemetry-js": "^1.5.0",
     "flatpickr": "4.6.13",
     "tslib": "2.3.0"
   }

src/package.json

@@ -72,10 +72,12 @@ export class Tooltip extends PopoverContainer implements AfterContentChecked {
 	/**
 	 * Optional data for templates passed as implicit context
 	 */
-	@Input() templateContext: any
+	@Input() templateContext: any;
 
 	@ViewChild("contentWrapper") wrapper: ElementRef<HTMLSpanElement>;
 
+	private timeoutId: any; // it should be number, but setTimeout below is matching the NodeJs type instead of the JS type
+
 	constructor(
 		protected elementRef: ElementRef,
 		protected ngZone: NgZone,
@@ -89,14 +91,20 @@ export class Tooltip extends PopoverContainer implements AfterContentChecked {
 
 	@HostListener("mouseenter", ["$event"])
 	mouseenter(event) {
-		setTimeout(() => {
+		// If a mouseleave is triggered before the tooltip is displayed (before setTimeout of mouseenter completes)
+		// we trigger the mouseleave only avoiding having to unecessary show the tooltip
+		clearTimeout(this.timeoutId);
+		this.timeoutId = setTimeout(() => {
 			this.handleChange(true, event);
 		}, this.enterDelayMs);
 	}
 
 	@HostListener("mouseleave", ["$event"])
 	mouseleave(event) {
-		setTimeout(() => {
+		// If a mouseleave is triggered before the tooltip is displayed (before setTimeout of mouseenter completes)
+		// we trigger the mouseleave only avoiding having to unecessary show the tooltip
+		clearTimeout(this.timeoutId);
+		this.timeoutId = setTimeout(() => {
 			this.handleChange(false, event);
 		}, this.leaveDelayMs);
 	}