https://github.com/kimmelsg/cj-js-styleguide
Quick reference styling react components / building JS projects
https://github.com/kimmelsg/cj-js-styleguide
Last synced: 8 months ago
JSON representation
Quick reference styling react components / building JS projects
- Host: GitHub
- URL: https://github.com/kimmelsg/cj-js-styleguide
- Owner: kimmelsg
- Created: 2016-01-08T21:35:48.000Z (almost 10 years ago)
- Default Branch: master
- Last Pushed: 2016-01-08T21:52:22.000Z (almost 10 years ago)
- Last Synced: 2025-01-12T06:43:06.344Z (10 months ago)
- Homepage:
- Size: 5.86 KB
- Stars: 1
- Watchers: 5
- Forks: 1
- Open Issues: 0
-
Metadata Files:
- Readme: README.md
Awesome Lists containing this project
README
# NavJobs JS Styleguide
*Majorly inspired by [airbnb](https://github.com/airbnb/javascript/tree/master/react)*
## Table of Contents
1. [React](#react)
1. [Basic Rules](#basic-rules)
1. [Class vs `React.createClass`](#class-vs-reactcreateclass)
1. [Naming](#naming)
1. [Declaration](#declaration)
1. [Alignment](#alignment)
1. [Quotes](#quotes)
1. [Spacing](#spacing)
1. [Props](#props)
1. [Parentheses](#parentheses)
1. [Tags](#tags)
1. [Methods](#methods)
1. [Ordering](#ordering)
1. [`isMounted`](#ismounted)
## Basic Rules
- When using [Immutable](https://facebook.github.io/immutable-js/) never use `.toJS()`
- no semicolons
- use ternary operators sparingly unless inside of `React.render`
- one React component per file
- only use `props` if building a child component, unless absolutely neccesary.
#React
## Class vs `React.createClass`
- Use stateless components when you can, otherwise `class extends React.Component`.
eslint rules: [`react/prefer-es6-class`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/prefer-es6-class.md).
```javascript
class Listing extends React.Component {
handleClick() {
}
render() {
return
}
}
//or
const Test = ({ title }) => (
{title}
)
```
## Naming
- **Extensions**: Use `.jsx` extension for React components.
- **Filename**: Use PascalCase for filenames. E.g., `ReservationCard.jsx`.
- **Reference Naming**: Use PascalCase for React components and camelCase for their instances.
eslint rules: [`react/jsx-pascal-case`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-pascal-case.md).
```javascript
// bad
import reservationCard from './ReservationCard'
// good
import ReservationCard from './ReservationCard'
// bad
const ReservationItem =
// good
const reservationItem =
```
- **Component Naming**: Use the filename as the component name. For example, `ReservationCard.jsx` should have a reference name of `ReservationCard`. However, for root components of a directory, use `index.jsx` as the filename and use the directory name as the component name:
```javascript
// bad
import Footer from './Footer/Footer'
// bad
import Footer from './Footer/index'
// good
import Footer from './Footer'
```
## Declaration
- Do not use `displayName` for naming components. Instead, name the component by reference.
```javascript
// bad
export default React.createClass({
displayName: 'ReservationCard',
// stuff goes here
})
// good
export default class ReservationCard extends React.Component {
}
```
## Alignment
- Follow these alignment styles for JSX syntax
eslint rules: [`react/jsx-closing-bracket-location`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-closing-bracket-location.md).
```javascript
// bad
// good
// if props fit in one line then keep it on the same line
// children get indented normally
```
## Quotes
- Always use double quotes (`"`) for JSX attributes, but single quotes for all other JS.
> Why? JSX attributes [can't contain escaped quotes](http://eslint.org/docs/rules/jsx-quotes), so double quotes make conjunctions like `"don't"` easier to type.
> Regular HTML attributes also typically use double quotes instead of single, so JSX attributes mirror this convention.
eslint rules: [`jsx-quotes`](http://eslint.org/docs/rules/jsx-quotes).
```javascript
// bad
// good
// bad
// good
```
## Spacing
- Always include a single space in your self-closing tag.
```javascript
// bad
// very bad
// bad
// good
```
## Props
- Always use camelCase for prop names.
```javascript
// bad
// good
```
- Omit the value of the prop when it is explicitly `true`.
eslint rules: [`react/jsx-boolean-value`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-boolean-value.md).
```javascript
// bad
// good
```
## Parentheses
- Wrap JSX tags in parentheses when they span more than one line.
eslint rules: [`react/wrap-multilines`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/wrap-multilines.md).
```javascript
// bad
render() {
return
}
// good
render() {
return (
)
}
// good, when single line
render() {
const body =
hello
return {body}
}
```
## Tags
- Always self-close tags that have no children.
eslint rules: [`react/self-closing-comp`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/self-closing-comp.md).
```javascript
// bad
// good
```
- If your component has multi-line properties, close its tag on a new line.
eslint rules: [`react/jsx-closing-bracket-location`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-closing-bracket-location.md).
```javascript
// bad
// good
```
## Methods
- Bind event handlers for the render method in the constructor.
> Why? A bind call in a the render path creates a brand new function on every single render.
eslint rules: [`react/jsx-no-bind`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-no-bind.md).
```javascript
// bad
class extends React.Component {
onClickDiv() {
// do stuff
}
render() {
return
}
}
// good
class extends React.Component {
constructor(props) {
super(props)
this.onClickDiv = this.onClickDiv.bind(this)
}
onClickDiv() {
// do stuff
}
render() {
return
}
}
```
- Do not use underscore prefix for internal methods of a React component.
```javascript
// bad
React.createClass({
_onClickSubmit() {
// do stuff
},
// other stuff
})
// good
class extends React.Component {
onClickSubmit() {
// do stuff
}
// other stuff
}
```
## Ordering
- Ordering for `class extends React.Component`:
1. `constructor`
1. optional `static` methods
1. `getChildContext`
1. `componentWillMount`
1. `componentDidMount`
1. `componentWillReceiveProps`
1. `shouldComponentUpdate`
1. `componentWillUpdate`
1. `componentDidUpdate`
1. `componentWillUnmount`
1. *clickHandlers or eventHandlers* like `onClickSubmit()` or `onChangeDescription()`
1. *getter methods for `render`* like `getSelectReason()` or `getFooterContent()`
1. *Optional render methods* like `renderNavigation()` or `renderProfilePicture()`
1. `render`
- How to define `propTypes`, `defaultProps`, `contextTypes`, etc...
```javascript
import React, { PropTypes } from 'react'
const propTypes = {
id: PropTypes.number.isRequired,
url: PropTypes.string.isRequired,
text: PropTypes.string,
}
const defaultProps = {
text: 'Hello World',
}
class Link extends React.Component {
static methodsAreOk() {
return true
}
render() {
return {this.props.text}
}
}
Link.propTypes = propTypes
Link.defaultProps = defaultProps
export default Link
```
eslint rules: [`react/sort-comp`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/sort-comp.md).
## `isMounted`
- Do not use `isMounted`.
> Why? [`isMounted` is an anti-pattern][anti-pattern], is not available when using ES6 classes, and is on its way to being officially deprecated.
[anti-pattern]: https://facebook.github.io/react/blog/2015/12/16/ismounted-antipattern.html
eslint rules: [`react/no-is-mounted`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-is-mounted.md).
**[⬆ back to top](#table-of-contents)**