Remove @TODOs I don't care about

This commit is contained in:
Benjamin Goering
2017-02-28 20:09:24 +08:00
parent d9df38fc3a
commit 97ebb66f8c
2 changed files with 1 additions and 5 deletions
-1
View File
@@ -108,7 +108,6 @@ class Comment extends React.Component {
const flag = getActionSummary('FlagActionSummary', comment);
const dontagree = getActionSummary('DontAgreeActionSummary', comment);
// @TODO(bengo) Would be best to only create these funcs on prop change
const addBestTag = () => addCommentTag({
id: comment.id,
tag: BEST_TAG,
+1 -4
View File
@@ -89,10 +89,7 @@ export class BestButton extends Component {
}
render() {
// @TODO(bengo) Consider adding the comment__action classes to other buttons to add cursor:pointer and never wrap the icons
// @TODO(bengo) Should I reuse another element like coral-ui button? Just doing what LikeButton does for now
// Oh. I think that's styled for the admin. Don't use coral-ui button until the whole comment bottom bar does.
// @TODO(bengo) Consider adding the comment__action--cursor-pointer to all buttons to add cursor:pointer and never wrap the icons
const {isBest, addBest, removeBest} = this.props;
const {isSaving} = this.state;
const disabled = isSaving || ! (isBest ? removeBest : addBest);