fix: opt for app ref tick call inside dropdown list

Author: SMassola

src/dropdown/dropdown.component.ts

@@ -357,7 +357,6 @@ export class Dropdown implements OnInit, AfterContentInit, AfterViewInit, OnDest
 		protected elementRef: ElementRef,
 		protected i18n: I18n,
 		protected dropdownService: DropdownService,
-		protected ngZone: NgZone,
 		protected elementService: ElementService) {}
 
 	/**
@@ -387,36 +386,34 @@ export class Dropdown implements OnInit, AfterContentInit, AfterViewInit, OnDest
 		const isUpdate = event => event && event.isUpdate;
 
 		this.view.select.subscribe(event => {
-			this.ngZone.run(() => {
-				if (this.type === "single" && !isUpdate(event)) {
-					this.closeMenu();
-					if (event.item && event.item.selected) {
-						if (this.itemValueKey) {
-							this.propagateChange(event.item[this.itemValueKey]);
-						} else {
-							this.propagateChange(event.item);
-						}
+			if (this.type === "single" && !isUpdate(event)) {
+				this.closeMenu();
+				if (event.item && event.item.selected) {
+					if (this.itemValueKey) {
+						this.propagateChange(event.item[this.itemValueKey]);
 					} else {
-						this.propagateChange(null);
+						this.propagateChange(event.item);
 					}
+				} else {
+					this.propagateChange(null);
 				}
+			}
 
-				if (this.type === "multi" && !isUpdate(event)) {
-					// if we have a `value` selector and selected items map them appropriately
-					if (this.itemValueKey && this.view.getSelected()) {
-						const values = this.view.getSelected().map(item => item[this.itemValueKey]);
-						this.propagateChange(values);
-					// otherwise just pass up the values from `getSelected`
-					} else {
-						this.propagateChange(this.view.getSelected());
-					}
-				}
-				// only emit selected for "organic" selections
-				if (!isUpdate(event)) {
-					this.checkForReorder();
-					this.selected.emit(event);
+			if (this.type === "multi" && !isUpdate(event)) {
+				// if we have a `value` selector and selected items map them appropriately
+				if (this.itemValueKey && this.view.getSelected()) {
+					const values = this.view.getSelected().map(item => item[this.itemValueKey]);
+					this.propagateChange(values);
+				// otherwise just pass up the values from `getSelected`
+				} else {
+					this.propagateChange(this.view.getSelected());
 				}
-			});
+			}
+			// only emit selected for "organic" selections
+			if (!isUpdate(event)) {
+				this.checkForReorder();
+				this.selected.emit(event);
+			}
 		});
 	}
 

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

@@ -9,7 +9,8 @@ import {
 	ViewChild,
 	ElementRef,
 	ViewChildren,
-	QueryList
+	QueryList,
+	ApplicationRef
 } from "@angular/core";
 import { Observable, isObservable, Subscription, of } from "rxjs";
 import { first } from "rxjs/operators";
@@ -226,7 +227,7 @@ export class DropdownList implements AbstractDropdownView, AfterViewInit, OnDest
 	/**
 	 * Creates an instance of `DropdownList`.
 	 */
-	constructor(public elementRef: ElementRef, protected i18n: I18n) {}
+	constructor(public elementRef: ElementRef, protected i18n: I18n, protected appRef: ApplicationRef) {}
 
 	/**
 	 * Retrieves array of list items and index of the selected item after view has rendered.
@@ -574,6 +575,7 @@ export class DropdownList implements AbstractDropdownView, AfterViewInit, OnDest
 			this.index = this.displayItems.indexOf(item);
 			this.highlightedItem = this.getItemId(this.index);
 			this.doEmitSelect(false);
+			this.appRef.tick();
 		}
 	}