Skip to content

Commit

Permalink
Add ESLint rule (object-curly-spacing) (mastodon#3498)
Browse files Browse the repository at this point in the history
  • Loading branch information
ykzts authored and koteitan committed Jun 25, 2017
1 parent 82e97da commit ae541f5
Show file tree
Hide file tree
Showing 9 changed files with 19 additions and 16 deletions.
3 changes: 3 additions & 0 deletions .eslintrc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,9 @@ rules:
comma-dangle:
- error
- always-multiline
object-curly-spacing:
- error
- always

react/jsx-wrap-multilines: error
react/jsx-no-bind: error
Expand Down
2 changes: 1 addition & 1 deletion app/javascript/mastodon/components/avatar_overlay.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ class AvatarOverlay extends React.PureComponent {
};

render() {
const {staticSrc, overlaySrc} = this.props;
const { staticSrc, overlaySrc } = this.props;

const baseStyle = {
backgroundImage: `url(${staticSrc})`,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,21 +46,21 @@ class EmojiPickerDropdown extends React.PureComponent {
}

onShowDropdown = () => {
this.setState({active: true});
this.setState({ active: true });
if (!EmojiPicker) {
this.setState({loading: true});
this.setState({ loading: true });
import(/* webpackChunkName: "emojione_picker" */ 'emojione-picker').then(TheEmojiPicker => {
EmojiPicker = TheEmojiPicker.default;
this.setState({loading: false});
this.setState({ loading: false });
}).catch(err => {
// TODO: show the user an error?
this.setState({loading: false});
this.setState({ loading: false });
});
}
}

onHideDropdown = () => {
this.setState({active: false});
this.setState({ active: false });
}

render () {
Expand Down
4 changes: 2 additions & 2 deletions app/javascript/mastodon/features/getting_started/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ import ImmutablePureComponent from 'react-immutable-pure-component';
const messages = defineMessages({
heading: { id: 'getting_started.heading', defaultMessage: 'Getting started' },
public_timeline: { id: 'navigation_bar.public_timeline', defaultMessage: 'Federated timeline' },
navigation_subheading: { id: 'column_subheading.navigation', defaultMessage: 'Navigation'},
settings_subheading: { id: 'column_subheading.settings', defaultMessage: 'Settings'},
navigation_subheading: { id: 'column_subheading.navigation', defaultMessage: 'Navigation' },
settings_subheading: { id: 'column_subheading.settings', defaultMessage: 'Settings' },
community_timeline: { id: 'navigation_bar.community_timeline', defaultMessage: 'Local timeline' },
preferences: { id: 'navigation_bar.preferences', defaultMessage: 'Preferences' },
follow_requests: { id: 'navigation_bar.follow_requests', defaultMessage: 'Follow requests' },
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,11 @@ class ImageLoader extends React.PureComponent {

loadImage(src) {
const image = new Image();
image.onerror = () => this.setState({loading: false, error: true});
image.onload = () => this.setState({loading: false, error: false});
image.onerror = () => this.setState({ loading: false, error: true });
image.onload = () => this.setState({ loading: false, error: false });
image.src = src;
this.lastSrc = src;
this.setState({loading: true});
this.setState({ loading: true });
}

render() {
Expand Down
4 changes: 2 additions & 2 deletions app/javascript/mastodon/features/ui/components/media_modal.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,11 @@ class MediaModal extends ImmutablePureComponent {
};

handleNextClick = () => {
this.setState({ index: (this.getIndex() + 1) % this.props.media.size});
this.setState({ index: (this.getIndex() + 1) % this.props.media.size });
}

handlePrevClick = () => {
this.setState({ index: (this.getIndex() - 1) % this.props.media.size});
this.setState({ index: (this.getIndex() - 1) % this.props.media.size });
}

handleKeyUp = (e) => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ const PageSix = ({ admin, domain }) => {
<p>
<FormattedMessage id='onboarding.page_six.admin' defaultMessage="Your instance's admin is {admin}." values={{ admin: <Permalink href={admin.get('url')} to={`/accounts/${admin.get('id')}`}>@{admin.get('acct')}</Permalink> }} />
<br />
<FormattedMessage id='onboarding.page_six.read_guidelines' defaultMessage="Please read {domain}'s {guidelines}!" values={{domain, guidelines: <a href='/about/more' target='_blank'><FormattedMessage id='onboarding.page_six.guidelines' defaultMessage='community guidelines' /></a> }}/>
<FormattedMessage id='onboarding.page_six.read_guidelines' defaultMessage="Please read {domain}'s {guidelines}!" values={{ domain, guidelines: <a href='/about/more' target='_blank'><FormattedMessage id='onboarding.page_six.guidelines' defaultMessage='community guidelines' /></a> }}/>
</p>
);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ const mapDispatchToProps = (dispatch, { type, id }) => ({
onScrollToBottom: debounce(() => {
dispatch(scrollTopTimeline(type, false));
dispatch(expandTimeline(type, id));
}, 300, {leading: true}),
}, 300, { leading: true }),

onScrollToTop: debounce(() => {
dispatch(scrollTopTimeline(type, true));
Expand Down
2 changes: 1 addition & 1 deletion config/webpack/translationRunner.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ const validateLanguages = function (languages, validators) {

if (invalidLanguages.length > 0) {
console.log(`\nError: Specified invalid LANGUAGES:`);
for (let {language, error} of invalidLanguages) {
for (let { language, error } of invalidLanguages) {
console.error(`* ${language}: ${error}`);
}
console.log(`\nUse yarn "manage:translations -- --help" for usage information\n`);
Expand Down

0 comments on commit ae541f5

Please sign in to comment.