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

Rename data table #1

Closed
wants to merge 3 commits into from
Closed

Rename data table #1

wants to merge 3 commits into from

Conversation

vpicone
Copy link

@vpicone vpicone commented Mar 28, 2019

Directed towards upstream PR: carbon-design-system#2208

Renames all instances of data-table-v2 to data-table

@asudoh
Copy link

asudoh commented Mar 28, 2019

@shinytoyrobots @alisonjoseph Do you think it's worth the risk? Another risk is doing the rename in v11. Thanks!

@lovemecomputer
Copy link

@asudoh i think this rename will lead to more consistent usage and adoption, and will make for an easier transition for project teams when migrating from v10 to v11 in the future. (and hopefully from then on?). unless the short timespan is a concern, i think we should do it

@asudoh
Copy link

asudoh commented Mar 29, 2019

@lovemecomputer You are right that we are talking about the short timespan. @vpicone Do you want to rebase BTW?

@vpicone vpicone closed this Mar 29, 2019
joshblack pushed a commit that referenced this pull request May 2, 2019
carbon-design-system#1702)

* fix: Removed the role of button from the expandable tile, resolves carbon-design-system#1583.

* fix: removed the tab index from the div of the expandable tile, resolves 1702.
joshblack added a commit that referenced this pull request Jun 17, 2019
* chore(project): update LICENSE

* chore(project): setup project files

* chore(project): add github files
joshblack added a commit that referenced this pull request Jun 17, 2019
* chore(project): update LICENSE

* chore(project): setup project files

* chore(project): add github files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants