From 594a238efb389805427c07d788558816b3240467 Mon Sep 17 00:00:00 2001 From: Jonas Dyrlie Date: Wed, 22 Jan 2025 08:57:34 +0100 Subject: [PATCH] Remove extra label and fix test --- .../OverrideFields/EmptyTextField.tsx | 31 +++++++------------ .../Override/Summary2Override.test.tsx | 8 ++--- 2 files changed, 15 insertions(+), 24 deletions(-) diff --git a/frontend/packages/ux-editor/src/components/config/componentSpecificContent/Summary2/Override/OverrideFields/EmptyTextField.tsx b/frontend/packages/ux-editor/src/components/config/componentSpecificContent/Summary2/Override/OverrideFields/EmptyTextField.tsx index 09143823cfc..85626d62971 100644 --- a/frontend/packages/ux-editor/src/components/config/componentSpecificContent/Summary2/Override/OverrideFields/EmptyTextField.tsx +++ b/frontend/packages/ux-editor/src/components/config/componentSpecificContent/Summary2/Override/OverrideFields/EmptyTextField.tsx @@ -1,12 +1,7 @@ import React, { type ChangeEvent } from 'react'; import classes from './EmptyTextField.module.css'; import type { Summary2OverrideConfig } from 'app-shared/types/ComponentSpecificConfig'; -import { - StudioAlert, - StudioLabelAsParagraph, - StudioProperty, - StudioTextfield, -} from '@studio/components'; +import { StudioAlert, StudioProperty, StudioTextfield } from '@studio/components'; import { useTranslation } from 'react-i18next'; type EmptyTextFieldProps = { @@ -38,19 +33,15 @@ export const EmptyTextField = ({ onChange, override }: EmptyTextFieldProps) => { } return ( - <> - - {t('ux_editor.component_properties.summary.override.empty_field_text')} - - setOpen(false)} - onKeyDown={({ key }) => key === 'Enter' && setOpen(false)} - value={override.emptyFieldText} - onChange={(event: ChangeEvent) => - onChange({ ...override, emptyFieldText: event.target.value }) - } - /> - + setOpen(false)} + onKeyDown={({ key }) => key === 'Enter' && setOpen(false)} + value={override.emptyFieldText} + onChange={(event: ChangeEvent) => + onChange({ ...override, emptyFieldText: event.target.value }) + } + /> ); }; diff --git a/frontend/packages/ux-editor/src/components/config/componentSpecificContent/Summary2/Override/Summary2Override.test.tsx b/frontend/packages/ux-editor/src/components/config/componentSpecificContent/Summary2/Override/Summary2Override.test.tsx index 20f4279cb52..e3aeee37cc3 100644 --- a/frontend/packages/ux-editor/src/components/config/componentSpecificContent/Summary2/Override/Summary2Override.test.tsx +++ b/frontend/packages/ux-editor/src/components/config/componentSpecificContent/Summary2/Override/Summary2Override.test.tsx @@ -322,13 +322,13 @@ describe('Summary2Override', () => { await userEvent.click(overrideCollapsedButton(1)); const emptyFieldText = 'asdf;ljr%'; const textFieldButton = screen.getByRole('button', { - name: textMock('ux_editor.component_properties.summary.override.empty_field_text'), + name: /ux_editor.component_properties.summary.override.empty_field_text/i, }); await user.click(textFieldButton); await user.type( - screen.getByLabelText( - textMock('ux_editor.component_properties.summary.override.empty_field_text'), - ), + screen.getByRole('textbox', { + name: /ux_editor.component_properties.summary.override.empty_field_text/i, + }), emptyFieldText, ); await waitFor(() =>