work for the #8493 (#8497)

surveyjs · Jul 2, 2024 · 22aa156 · 22aa156
1 parent c7aa246
commit 22aa156
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/dropdownListModel.ts b/src/dropdownListModel.ts
@@ -422,7 +422,7 @@ export class DropdownListModel extends Base {
   }
 
   public onClick(event: any): void {
-    if (this.question.readOnly || this.question.isDesignMode || this.question.isPreviewStyle) return;
+    if (this.question.readOnly || this.question.isDesignMode || this.question.isPreviewStyle || this.question.isReadOnlyAttr) return;
     this._popupModel.toggleVisibility();
     this.focusItemOnClickAndPopup();
     this.question.focus();

diff --git a/tests/dropdown_list_model_test.ts b/tests/dropdown_list_model_test.ts
@@ -904,6 +904,19 @@ QUnit.test("Dropdown should not be open on click in preview mode", (assert) => {
   assert.notOk(popupModel.isVisible);
 });
 
+QUnit.test("Dropdown should not be open on click in display mode", (assert) => {
+  const survey = new SurveyModel(jsonDropdown);
+  survey.mode = "display";
+
+  const question = <QuestionDropdownModel>survey.getAllQuestions()[0];
+
+  const dropdownListModel = question.dropdownListModel;
+  const popupModel = dropdownListModel.popupModel;
+  assert.notOk(popupModel.isVisible);
+  dropdownListModel.onClick(new Event("click"));
+  assert.notOk(popupModel.isVisible);
+});
+
 QUnit.test("order & locale change", function (assert) {
   const survey = new SurveyModel({ elements: [
     { type: "dropdown", name: "q1", choicesOrder: "asc",