Skip to content

chore: typescript types for props in TableBody#13982

Merged
kodiakhq[bot] merged 7 commits into
carbon-design-system:mainfrom
imp-dance:12521-tablebody-types
Jun 15, 2023
Merged

chore: typescript types for props in TableBody#13982
kodiakhq[bot] merged 7 commits into
carbon-design-system:mainfrom
imp-dance:12521-tablebody-types

Conversation

@imp-dance

Copy link
Copy Markdown
Contributor

Closes #12521

Adds Typescript definitions for TableBody props.

Changelog

  • TableBody.js -> TableBody.tsx
  • Added types for TableBody component's props

Additional comments

Was not sure if I should keep the docstring for aria-live. The inherited docstring from HTMLAttributes<HTMLTableSectionElement> looks like this:

Indicates that an element will be updated, and describes the types of updates the user agents, assistive technologies, and user can expect from the live region.

@imp-dance imp-dance requested a review from a team as a code owner June 13, 2023 07:14
@netlify

netlify Bot commented Jun 13, 2023

Copy link
Copy Markdown

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 8bcef52
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/6488174036b0120008465545
😎 Deploy Preview https://deploy-preview-13982--carbon-components-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify

netlify Bot commented Jun 13, 2023

Copy link
Copy Markdown

Deploy Preview for carbon-elements ready!

Name Link
🔨 Latest commit 8bcef52
🔍 Latest deploy log https://app.netlify.com/sites/carbon-elements/deploys/64881740ee3e5c0008d0c308
😎 Deploy Preview https://deploy-preview-13982--carbon-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify

netlify Bot commented Jun 13, 2023

Copy link
Copy Markdown

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit a036397
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/648b1e6a2b469a0008f08ace
😎 Deploy Preview https://deploy-preview-13982--carbon-components-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify

netlify Bot commented Jun 13, 2023

Copy link
Copy Markdown

Deploy Preview for carbon-elements ready!

Name Link
🔨 Latest commit a036397
🔍 Latest deploy log https://app.netlify.com/sites/carbon-elements/deploys/648b1e6b1fa2040008102945
😎 Deploy Preview https://deploy-preview-13982--carbon-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@francinelucca francinelucca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! 🚀

@kodiakhq kodiakhq Bot merged commit e7800e7 into carbon-design-system:main Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add TypeScript types to TableBody

3 participants