Generally we follow the Airbnb React Style Guide.
- Try to keep files small and focused.
- Break large components up into sub-components.
- Components and types that needs to be used by external plugins needs to go into @grafana/ui
- Components should get their own folder under features/xxx/components
- Sub components can live in that component folders, so small component do not need their own folder
- Place test next to their component file (same dir)
- Component sass should live in the same folder as component code
- State logic & domain models should live in features/xxx/state
- Containers (pages) can live in feature root features/xxx
- up for debate?
- Name callback props and handlers with an "on" prefix.
// good
onChange = () => {
};
render() {
return (
<MyComponent onChange={this.onChange} />
);
}
// bad
handleChange = () => {
};
render() {
return (
<MyComponent changed={this.handleChange} />
);
}
- React Component definitions
// good
export class YourClass extends PureComponent<{},{}> { ... }
// bad
export class YourClass extends PureComponent { ... }
- React Component constructor
// good
constructor(props:Props) {...}
// bad
constructor(props) {...}
- React Component defaultProps
// good
static defaultProps: Partial<Props> = { ... }
// bad
static defaultProps = { ... }
- Don't mutate state in reducers or thunks.
- Use helpers
actionCreatorFactory
andreducerFactory
instead of traditionalswitch statement
reducers in Redux. See Redux framework for more details. - Use
reducerTester
to test reducers. See Redux framework for more details. - Use state selectors to access state instead of accessing state directly.