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(data-table): reset expand button height in short table #3057

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented Jun 13, 2019

Closes #3053

The expand button height for expandable data tables was identical for all data table variants, this PR unsets the height for short data tables

Testing / Reviewing

Ensure data table variants appear correct

@emyarod emyarod added this to the v10.4 milestone Jun 13, 2019
@emyarod emyarod added this to PR: needs review 🕵️ in Carbon support via automation Jun 13, 2019
@netlify
Copy link

netlify bot commented Jun 13, 2019

Deploy preview for the-carbon-components ready!

Built with commit 107a14d

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

@netlify
Copy link

netlify bot commented Jun 13, 2019

Deploy preview for carbon-elements ready!

Built with commit 107a14d

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

@netlify
Copy link

netlify bot commented Jun 13, 2019

Deploy preview for carbon-components-react ready!

Built with commit 107a14d

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

Copy link
Contributor

@asudoh asudoh left a comment

Choose a reason for hiding this comment

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

LGTM 👍 - Thanks @emyarod!

@emyarod emyarod changed the title fix(data-table): unset expand button height in short table fix(data-table): reset expand button height in short table Jun 14, 2019
Carbon support automation moved this from PR: needs review 🕵️ to PR: ready to merge 🔜 Jun 17, 2019
@asudoh asudoh merged commit deba49b into carbon-design-system:master Jun 18, 2019
Carbon support automation moved this from PR: ready to merge 🔜 to Issue/PR: Closed 🙌 Jun 18, 2019
@emyarod emyarod deleted the 3053-expandable-short-data-table-icon-positioning branch June 18, 2019 12:50
cal-smith pushed a commit to cal-smith/carbon-components that referenced this pull request Jun 21, 2019
cal-smith pushed a commit to cal-smith/carbon-components that referenced this pull request Jun 21, 2019
cal-smith pushed a commit to cal-smith/carbon-components that referenced this pull request Jun 21, 2019
@emyarod emyarod mentioned this pull request Jul 15, 2019
41 tasks
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.

Expandable Data Table icon positioning: wrong position in short tables
3 participants