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

[TableCell] Add type property #9852

Merged
merged 3 commits into from Jan 12, 2018
Merged

[TableCell] Add type property #9852

merged 3 commits into from Jan 12, 2018

Conversation

kgregory
Copy link
Member

@kgregory kgregory commented Jan 12, 2018

This new property can be used to style a TableCell as if it is part of a TableHead, TableBody, or TableFooter component.

As part of this enhancement, the styles defined in TableHead and TableFooter have been removed and added to their respective classes in TableCell.

Breaking Changes

  • TableHead, TableBody and TableFooter no longer offer a CSS API, which means their root classes are no longer available. To style the root element in these components, a className prop can be passed, as all non-API props will be spread to the root element.

@oliviertassinari oliviertassinari self-assigned this Jan 12, 2018
Ken Gregory and others added 3 commits January 12, 2018 23:07
This new property can be used to get a TableCell to be styled as if it was part of a TableHead, TableBody, or TableFooter component.

As part of this enhancement, the styles defined in TableHead and TableFooter have been removed and added to their respective classes in TableCell.

**Breaking Changes:**

- TableHead and TableFooter no longer offer a CSS API, which means their root classes are no longer available.  To style this element, a `className` prop can be passed, as all non-API props will be spread to the root element.
@oliviertassinari oliviertassinari added component: table This is the name of the generic UI component, not the React module! breaking change labels Jan 12, 2018
@oliviertassinari
Copy link
Member

@kgregory Awesome job! It's a great step toward higher customizability.

@oliviertassinari oliviertassinari changed the title [TableCell] Adding type property [TableCell] Add type property Jan 12, 2018
@oliviertassinari oliviertassinari merged commit c4c5b19 into mui:v1-beta Jan 12, 2018
@ljvanschie
Copy link
Contributor

The class names for table cells have been changed (head -> typeHead, new keys typeBody and typeFooter), but TableCellClassKey in TableCell.d.ts hasn't been updated. Can I just create a PR to fix it or should I create an issue first?

@oliviertassinari
Copy link
Member

@ljvanschie A pull-request would be great!

@kgregory kgregory deleted the feature/tablecell-type branch March 5, 2018 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change component: table This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants