Skip to content
This repository has been archived by the owner on Dec 13, 2020. It is now read-only.

fix #215 - added shortcut for change doc status to complete #304

Merged
merged 2 commits into from
Feb 15, 2017

Conversation

bambeusz
Copy link
Contributor

No description provided.

import React, { Component } from 'react';
import { Shortcuts } from 'react-shortcuts';

class PaginationContextShortcuts extends Component {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chmieludaniel wrong class name.

}

handleChangeStatus = (status) => {
this.props.onChange(status);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you are changing some file and it is out of our code schema, please fix it too beside.

const {onChange} = ...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and whats more, here i believe you can use const instead of let

@@ -132,8 +146,13 @@ class ActionButton extends Component {
<div className={"tag tag-" + this.getStatusContext(abrev)}>{value} </div>
<i className={"meta-icon-chevron-1 meta-icon-" + this.getStatusContext(abrev)} />
<ul className="dropdown-status-list">
{this.renderStatusList(this.state.list)}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thats what im asking, thank you.

@damianprzygodzki damianprzygodzki merged commit 15574f3 into master Feb 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants