You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by el...@apache.org on 2026/07/15 22:03:46 UTC

(superset) branch master updated: fix(a11y): associate Description label with TagModal input (#42034)

This is an automated email from the ASF dual-hosted git repository.

eschutho pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new da518d7a107 fix(a11y): associate Description label with TagModal input (#42034)
da518d7a107 is described below

commit da518d7a1071cce8e8169155c3e29efceb78c1f7
Author: Elizabeth Thompson <es...@gmail.com>
AuthorDate: Wed Jul 15 15:03:40 2026 -0700

    fix(a11y): associate Description label with TagModal input (#42034)
---
 superset-frontend/src/features/tags/TagModal.test.tsx | 5 +++++
 superset-frontend/src/features/tags/TagModal.tsx      | 5 ++++-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/superset-frontend/src/features/tags/TagModal.test.tsx b/superset-frontend/src/features/tags/TagModal.test.tsx
index 0fce2de2e83..ee38ff43ecb 100644
--- a/superset-frontend/src/features/tags/TagModal.test.tsx
+++ b/superset-frontend/src/features/tags/TagModal.test.tsx
@@ -45,6 +45,11 @@ test('renders correctly in create mode', () => {
   expect(getByText('Create Tag')).toBeInTheDocument();
 });
 
+test('description input has an accessible name', async () => {
+  render(<TagModal {...mockedProps} />);
+  expect(await screen.findByLabelText('Tag description')).toBeInTheDocument();
+});
+
 test('renders correctly in edit mode', () => {
   fetchMock.get(fetchEditFetchObjects, [[]]);
   const editTag: Tag = {
diff --git a/superset-frontend/src/features/tags/TagModal.tsx b/superset-frontend/src/features/tags/TagModal.tsx
index aea360617de..1d80276d453 100644
--- a/superset-frontend/src/features/tags/TagModal.tsx
+++ b/superset-frontend/src/features/tags/TagModal.tsx
@@ -312,8 +312,11 @@ const TagModal: FC<TagModalProps> = ({
         </Flex>
 
         <Flex vertical gap={theme.sizeUnit}>
-          <FormLabel>{t('Description')}</FormLabel>
+          <FormLabel htmlFor="tag-description">
+            {t('Tag description')}
+          </FormLabel>
           <Input
+            id="tag-description"
             className="tag-input"
             onChange={handleDescriptionChange}
             placeholder={t('Add description of your tag')}