Selected options in Content Selector get deselected on dropdown scroll …

…#3305

- BaseRichComboBox: comboBox.addOption doesn't select item that is supposed to be selected
- DropdownList: navigating to the first selected row only if dropdown was not open yet
- no need to fetch a next batch if dropdown is already closed

src/main/resources/assets/admin/common/js/ui/selector/DropdownList.ts

@@ -117,16 +117,21 @@ export class DropdownList<OPTION_DISPLAY_VALUE> {
     }
 
     showDropdown(selectedOptions?: Option<OPTION_DISPLAY_VALUE>[], noOptionsText?: string) {
+        const isShown: boolean = this.isDropdownShown();
 
         if (this.hasOptions()) {
             this.emptyDropdown.hide();
             this.dropdownGrid.show();
             this.dropdownGrid.adjustGridHeight();
+
             if (selectedOptions) {
                 this.dropdownGrid.markSelections(selectedOptions);
                 if (selectedOptions.length > 0) {
                     this.dropdownGrid.markReadOnly(selectedOptions);
-                    this.navigateToRowIfNotActive(selectedOptions[0]);
+
+                    if (!isShown) {
+                        this.navigateToRowIfNotActive(selectedOptions[0]);
+                    }
                 }
             }
         } else {

src/main/resources/assets/admin/common/js/ui/selector/combobox/BaseRichComboBox.ts

@@ -366,7 +366,7 @@ export class BaseRichComboBox<OPTION_DATA_TYPE, LOADER_DATA_TYPE>
     }
 
     private loadNextRangeIfNeeded(containerEl: HTMLElement, handler: () => void): void {
-        if (this.isScrolledToBottom(containerEl)) {
+        if (this.isScrolledToBottom(containerEl) && this.comboBox.isDropdownShown()) {
             handler();
         }
     }
@@ -378,9 +378,10 @@ export class BaseRichComboBox<OPTION_DATA_TYPE, LOADER_DATA_TYPE>
 
     private handleRangeLoad(handler: () => void) {
         const viewportEl: HTMLElement = this.comboBox.getComboBoxDropdownGrid().getGrid().getViewportEl();
+        const loadHandler = () => this.loadNextRangeIfNeeded(viewportEl, handler);
 
-        viewportEl.addEventListener('scroll', () => this.loadNextRangeIfNeeded(viewportEl, handler));
-        this.onLoaded(() => this.loadNextRangeIfNeeded(viewportEl, handler));
+        viewportEl.addEventListener('scroll', loadHandler);
+        this.onLoaded(loadHandler);
     }
 
     private setupLoader(loader: BaseLoader<LOADER_DATA_TYPE>) {
@@ -418,26 +419,12 @@ export class BaseRichComboBox<OPTION_DATA_TYPE, LOADER_DATA_TYPE>
         }
     }
 
-    private handleLoadedData(event: LoadedDataEvent<LOADER_DATA_TYPE>) {
+    private handleLoadedData(event: LoadedDataEvent<LOADER_DATA_TYPE>): Q.Promise<void> {
         this.errorContainer.hide();
-        const optionCount: number = this.getOptionCount();
+
         return this.createOptions(event.getData().map(this.loadedItemToDisplayValue.bind(this))).then(
             (options: Option<OPTION_DATA_TYPE>[]) => {
-                let appendOptions: boolean = false;
-
-                if (event.isPostLoad() && optionCount > 0) {
-                    const lastOption: Option<OPTION_DATA_TYPE> = this.getOptionByRow(optionCount - 1);
-                    appendOptions = options.length > optionCount && options[optionCount - 1].getValue() === lastOption.getValue();
-                }
-
-                if (appendOptions) {
-                    for (let i: number = optionCount; i < options.length; i++) {
-                        this.comboBox.addOption(options[i]);
-                    }
-                } else {
-                    this.comboBox.setOptions(options, event.isPostLoad());
-                }
-
+                this.comboBox.setOptions(options, event.isPostLoad());
                 this.notifyLoaded(options.map((option) => option.getDisplayValue()), event.isPostLoad());
                 return;
             });