#7467 Dropdown With Lazy Loading Within a Dynamic Panel doesn't displ…

…ay search results on Mobile Fixes #7467
surveyjs · Dec 7, 2023 · 541cbdc · 541cbdc
1 parent c90aa0e
commit 541cbdc
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 1 deletion.
diff --git a/src/question_dropdown.ts b/src/question_dropdown.ts
@@ -295,7 +295,7 @@ export class QuestionDropdownModel extends QuestionSelectBase {
   protected onVisibleChoicesChanged(): void {
     super.onVisibleChoicesChanged();
 
-    if (this.popupModel) {
+    if (!this.isLoadingFromJson && this.popupModel) {
       this.dropdownListModel.updateItems();
     }
   }

diff --git a/tests/dropdown_list_model_test.ts b/tests/dropdown_list_model_test.ts
@@ -2,6 +2,7 @@ import { DropdownListModel } from "../src/dropdownListModel";
 import { ListModel } from "../src/list";
 import { PopupModel } from "../src/popup";
 import { QuestionDropdownModel } from "../src/question_dropdown";
+import { QuestionPanelDynamicModel } from "../src/question_paneldynamic";
 import { SurveyModel } from "../src/survey";
 import { _setIsTouch } from "../src/utils/devices";
 
@@ -890,4 +891,29 @@ QUnit.test("DropdownListModel filterReadOnly", (assert) => {
   assert.notOk(dropdownListModel.filterReadOnly);
   dropdownListModel.onBlur({ stopPropagation: () => { } });
   assert.ok(dropdownListModel.filterReadOnly);
+});
+QUnit.test("DropdownListModel in panel filterString change callback", (assert) => {
+  const survey = new SurveyModel({
+    elements: [
+      {
+        type: "paneldynamic",
+        name: "p",
+        templateElements: [
+          {
+            type: "dropdown",
+            name: "country",
+            choicesLazyLoadEnabled: true,
+            showOtherItem: true,
+          },
+        ],
+      },
+    ],
+    showQuestionNumbers: false,
+  });
+  const question = <QuestionPanelDynamicModel>survey.getAllQuestions()[0];
+  question.addPanel();
+
+  const dropdownListModel = (question.panels[0].elements[0] as QuestionDropdownModel).dropdownListModel;
+  dropdownListModel["listModel"].filterString = "abc";
+  assert.equal(dropdownListModel.filterString, "abc");
 });