Skip to content

Instantly share code, notes, and snippets.

@lelandrichardson
Last active February 15, 2016 20:34
Show Gist options
  • Save lelandrichardson/c30177543604b418e83a to your computer and use it in GitHub Desktop.
Save lelandrichardson/c30177543604b418e83a to your computer and use it in GitHub Desktop.
var styles = StyleSheet.create({
wrapper: {
// ...
},
text: {
// ...
}
})
var Button = React.createClass({
render() {
return (
<View style={[styles.wrapper, this.props.style]}>
<Text style={styles.text}>{this.props.children}</Text>
</View>
);
}
});
var MoreCustomButtonOptionOne = React.createClass({
render() {
return (
<View style={[styles.wrapper, this.props.style]}>
<Text style={[styles.text, this.props.textStyle]}>{this.props.children}</Text>
</View>
);
}
})
var MoreCustomButtonOptionTwo = React.createClass({
render() {
var { color, fontSize, ...style } = this.props.style || {};
var textStyle = { color, fontSize };
return (
<View style={[styles.wrapper, style]}>
<Text style={[styles.text, textStyle]}>{this.props.children}</Text>
</View>
);
}
})
@gaearon
Copy link

gaearon commented May 12, 2015

Our experience with a very visually heavy website with about a hundred of components told us that the props are the only sane way to scale. If you take shortcuts like using CSS cascade, it's so damn hard to move components around and change them later. You get all sorts of implicit dependencies. On the other hand, when props are the only contract to your component, you can even work on them in isolation. It feels so right, and once you try it, you're never going back to CSS cascade hell.

@gaearon
Copy link

gaearon commented May 12, 2015

One shortcut we've been taking is a context prop that is more “flexible”, e.g. <Button context='thatDamnPopupMenu'> that becomes .Button--isInThatDamnPopupMenu. This kinda couples the components, but sometimes there's no way to avoid coupling. Like if there's only one place in the app where button's styles are different (usually due to layout) and you can't make up a prop that makes more sense than a generic context. Still, it is absolutely explicit and defined inside Button, so this coupling is visible and colocated with the rest of the Button styles.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment