From 54141b32725c53cfd0f947d67bd363b76bac2f89 Mon Sep 17 00:00:00 2001 From: Leah Schumann Date: Wed, 21 Dec 2022 07:35:27 -0500 Subject: [PATCH 1/2] Removed 'disabledAutofill' prop from dropdown-question to match textbox-question and prevent the State label from being read incorrectly by JAWS --- src/common-components/dropdown-question/dropdown-question.vue | 1 - .../address-questions/address-questions.spec.js | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/src/common-components/dropdown-question/dropdown-question.vue b/src/common-components/dropdown-question/dropdown-question.vue index 4567994f1..e7c183e49 100644 --- a/src/common-components/dropdown-question/dropdown-question.vue +++ b/src/common-components/dropdown-question/dropdown-question.vue @@ -38,7 +38,6 @@ export default { }, isDisabled: Boolean, isRequired: Boolean, - disableAutoFill: Boolean, validationRules: String, cmsWidgetName: String, hasError: Boolean, diff --git a/src/layouts/address-lookup/customer-questions/address-questions/address-questions.spec.js b/src/layouts/address-lookup/customer-questions/address-questions/address-questions.spec.js index 439e2922c..c8d24c57e 100644 --- a/src/layouts/address-lookup/customer-questions/address-questions/address-questions.spec.js +++ b/src/layouts/address-lookup/customer-questions/address-questions/address-questions.spec.js @@ -128,7 +128,7 @@ describe("address-questions.vue", () => { expect(wrapper.vm.$loadScript).not.toHaveBeenCalled(); }); - test("address field is focused => disable autocomplete", async () => { + test("address field is focused => disable autofill", async () => { // Arrange let focusEventCallbackFunction; autocompleteElement.addEventListener = jest From 571fad279bdc01c17997e41d99f4b8877959dd9b Mon Sep 17 00:00:00 2001 From: Leah Schumann Date: Wed, 21 Dec 2022 07:52:38 -0500 Subject: [PATCH 2/2] Removed test for inserting ⁠ into the visual label --- .../dropdown-question.spec.js | 23 ------------------- 1 file changed, 23 deletions(-) diff --git a/src/common-components/dropdown-question/dropdown-question.spec.js b/src/common-components/dropdown-question/dropdown-question.spec.js index 249b00710..20bba1712 100644 --- a/src/common-components/dropdown-question/dropdown-question.spec.js +++ b/src/common-components/dropdown-question/dropdown-question.spec.js @@ -48,29 +48,6 @@ describe("dropdownQuestion.vue", () => { expect(label.text()).toContain(questionText); }); - it("Should render the 'questionText' data value with '⁠' after the first character of each word in the label text when disableAutoFill is true.", async () => { - // Arrange - const wrapper = shallowMount(dropdownQuestion, { - propsData: { - options: {}, - disableAutoFill: true, - }, - mixins: [mockMixin], - }); - - // Mock CMS content ... - // Trust me, the below instance of the string "Q⁠uestion Text" actually has the ⁠ in it. You just can't see it - // Don't believe me? Copy and paste it into Google. Then inspect the search field element in Dev Tools, - // you will see "Q⁠uestion T⁠ext" - const expectedQuestionText = "Q⁠uestion T⁠ext"; - - // Act - const label = wrapper.find("label"); - - // Assert - expect(label.text()).toContain(expectedQuestionText); - }); - it("Should return input id as the id of the select field", async () => { // Arrange const wrapper = shallowMount(dropdownQuestion, {