Skip to content

Commit

Permalink
[styles] Warn when the first argument is wrong (#11953)
Browse files Browse the repository at this point in the history
  • Loading branch information
oliviertassinari authored Jun 22, 2018
1 parent d412a85 commit 2386655
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
2 changes: 1 addition & 1 deletion docs/src/pages/customization/css-in-js/css-in-js.md
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,7 @@ For instance, it can be used to defined a `getInitialProps()` static method (nex

#### Arguments

1. `styles` (*Function | Object*): A function generating the styles or an object.
1. `styles` (*Function | Object*): A function generating the styles or a styles object.
It will be linked to the component.
Use the function signature if you need to have access to the theme. It's provided as the first argument.
2. `options` (*Object* [optional]):
Expand Down
8 changes: 8 additions & 0 deletions packages/material-ui/src/styles/getStylesCreator.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,14 @@ function arrayMerge(destination, source) {
function getStylesCreator(stylesOrCreator) {
const themingEnabled = typeof stylesOrCreator === 'function';

warning(
typeof stylesOrCreator === 'object' || themingEnabled,
[
'Material-UI: the first argument provided to withStyles() is invalid.',
'You need to provide a function generating the styles or a styles object.',
].join('\n'),
);

function create(theme, name) {
const styles = themingEnabled ? stylesOrCreator(theme) : stylesOrCreator;

Expand Down

0 comments on commit 2386655

Please sign in to comment.