-
Notifications
You must be signed in to change notification settings - Fork 16
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
Table: fix the empty table column cell element #725
Conversation
Squashed commit of the following: commit 9ef1e18 Author: Mat Harris <mat.harris@ithaka.org> Date: Fri Dec 8 12:06:10 2023 -0500 A11y revamp: Pharos buttons (non-breaking change) (ithaka#628) * docs: add @sirrah-tam as a contributor * feat(button): add a11y naming convention for aria * feat(button): add aria-description support * chore: add changset * Update packages/pharos/src/components/button/pharos-button.ts Co-authored-by: Dane Hillard <github@danehillard.com> * Update packages/pharos/src/components/button/pharos-button.ts Co-authored-by: Dane Hillard <github@danehillard.com> * feat(button): move property deprecated flag * fix(button): remove ts-ignore after lit upgrade * feat(button): add a11y state typing * fix: add TODO for future reference Co-authored-by: Dane Hillard <github@danehillard.com> * fix(a11y attributes): update AriaHiddenState name --------- Co-authored-by: Dane Hillard <github@danehillard.com>
🦋 Changeset detectedLatest commit: 882671a The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A unit test that looks at the empty column state and verifies it might be nice?
Going to close this PR and re-open with a clean commit history |
This change: (check at least one)
Is this a breaking change? (check one)
Is the: (complete all)
What does this change address?
Resolves #724
How does this change work?
Updates the
<tb></tb>
to<td></td>