Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(a11y): make tagName mandatory in Text and add it where necessary #715

Conversation

lismith2-cisco
Copy link
Collaborator

@lismith2-cisco lismith2-cisco commented Nov 26, 2024

Description

Make tagName mandatory in Text and add it where necessary. This is to ensure tag reflects page structure, not stying.

  • Text component
  • MeetingContainer component
  • component-conversions util

have changes to their interface, all other changes are trivial (although this touches a lot of files)

Links

https://jira-eng-gpk2.cisco.com/jira/browse/SPARK-582909

@lismith2-cisco lismith2-cisco added the validated If the pull request is validated automation. label Nov 26, 2024
@lismith2-cisco lismith2-cisco marked this pull request as ready for review November 26, 2024 16:47
@lismith2-cisco lismith2-cisco force-pushed the SPARK-582909-mandatory-text-tagname branch 2 times, most recently from e37aa86 to 5b320d7 Compare November 28, 2024 12:21
Copy link
Collaborator

@nataliadelmar nataliadelmar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, also tested on storybook nothing seems off

@nataliadelmar
Copy link
Collaborator

Let's first have a Cantina PR ready before we merge this otherwise it would block MRV2 bump

@lismith2-cisco lismith2-cisco force-pushed the SPARK-582909-mandatory-text-tagname branch from 6bc5b00 to 39f6250 Compare December 5, 2024 14:31
@lismith2-cisco lismith2-cisco force-pushed the SPARK-582909-mandatory-text-tagname branch from 39f6250 to a317a99 Compare December 11, 2024 10:17
@lismith2-cisco lismith2-cisco merged commit 1c0e4d7 into momentum-design:master Dec 11, 2024
5 checks passed
Copy link

🎉 This PR is included in version 26.184.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released validated If the pull request is validated automation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants