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

fix: duplicate id data-table in carbon-website #5080

Conversation

xxxle0
Copy link
Contributor

@xxxle0 xxxle0 commented Jan 17, 2020

No close issue in this package but relate to carbon-design-system/carbon-website#652 in carbon website because the duplicate id component in a same page

Changelog

New

  • {{new thing}}

Changed

  • {{change thing}}

Removed

  • {{removed thing}}

Testing / Reviewing

{{ Add descriptions, steps or a checklist for how reviewers can verify this PR works or not }}

@xxxle0 xxxle0 requested a review from a team as a code owner January 17, 2020 07:03
@ghost ghost requested review from asudoh and tw15egan January 17, 2020 07:03
@netlify
Copy link

netlify bot commented Jan 17, 2020

Deploy preview for the-carbon-components ready!

Built with commit 3dc71b4

https://deploy-preview-5080--the-carbon-components.netlify.com

@netlify
Copy link

netlify bot commented Jan 17, 2020

Deploy preview for carbon-elements ready!

Built with commit 3dc71b4

https://deploy-preview-5080--carbon-elements.netlify.com

@netlify
Copy link

netlify bot commented Jan 17, 2020

Deploy preview for carbon-components-react failed.

Built with commit 3dc71b4

https://app.netlify.com/sites/carbon-components-react/deploys/5e215c4cf56de3000965d84d

@netlify
Copy link

netlify bot commented Jan 17, 2020

Deploy preview for the-carbon-components ready!

Built with commit d6a5c44

https://deploy-preview-5080--the-carbon-components.netlify.com

@netlify
Copy link

netlify bot commented Jan 17, 2020

Deploy preview for carbon-elements ready!

Built with commit d6a5c44

https://deploy-preview-5080--carbon-elements.netlify.com

@netlify
Copy link

netlify bot commented Jan 17, 2020

Deploy preview for carbon-components-react ready!

Built with commit d6a5c44

https://deploy-preview-5080--carbon-components-react.netlify.com

Copy link
Member

@emyarod emyarod left a comment

Choose a reason for hiding this comment

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

looks good to me

Copy link
Member

@tw15egan tw15egan left a comment

Choose a reason for hiding this comment

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

👍

@joshblack joshblack merged commit 65cce52 into carbon-design-system:master Jan 17, 2020
joshblack added a commit to joshblack/carbon that referenced this pull request Jan 23, 2020
…5080)

* fix: duplicate id data-table in carbon-website

* fix: fix duplicate label of a rest of page

Co-authored-by: TJ Egan <tw15egan@gmail.com>
Co-authored-by: Josh Black <josh@josh.black>
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.

4 participants