rename nop handler to handleNoOp

This also adds the comment in action_bar.js to
status_action_bar.js, clarifying that a future
version could improve this code by modifying
EmojiPickerDropdown.
This commit is contained in:
fef 2022-12-02 01:00:08 +00:00 committed by Jeremy Kescher
parent 6ea8080771
commit adc1cd4823
No known key found for this signature in database
GPG key ID: 80A419A7A613DFA4
4 changed files with 8 additions and 8 deletions

View file

@ -207,7 +207,7 @@ class StatusActionBar extends ImmutablePureComponent {
this.props.onAddFilter(this.props.status);
};
nop = () => {}
handleNoOp = () => {} // hack for reaction add button
render () {
const { status, intl, withDismiss, withCounters, showReplyCount, scrollKey } = this.props;
@ -317,7 +317,7 @@ class StatusActionBar extends ImmutablePureComponent {
const reactButton = (
<IconButton
className='status__action-bar-button'
onClick={this.nop} // EmojiPickerDropdown handles that
onClick={this.handleNoOp} // EmojiPickerDropdown handles that
title={intl.formatMessage(messages.react)}
disabled={!canReact}
icon='plus'

View file

@ -144,7 +144,7 @@ class ActionBar extends React.PureComponent {
navigator.clipboard.writeText(url);
};
nop = () => {} // hack for reaction add button
handleNoOp = () => {} // hack for reaction add button
render () {
const { status, intl } = this.props;
@ -207,7 +207,7 @@ class ActionBar extends React.PureComponent {
const reactButton = (
<IconButton
className='plus-icon'
onClick={this.nop} // EmojiPickerDropdown handles that
onClick={this.handleNoOp} // EmojiPickerDropdown handles that
title={intl.formatMessage(messages.react)}
disabled={!canReact}
icon='plus'

View file

@ -242,7 +242,7 @@ class StatusActionBar extends ImmutablePureComponent {
this.props.onFilter();
};
nop = () => {}
handleNoOp = () => {} // hack for reaction add button
render () {
const { status, relationship, intl, withDismiss, withCounters, scrollKey } = this.props;
@ -374,7 +374,7 @@ class StatusActionBar extends ImmutablePureComponent {
const reactButton = (
<IconButton
className='status__action-bar-button'
onClick={this.nop} // EmojiPickerDropdown handles that
onClick={this.handleNoOp} // EmojiPickerDropdown handles that
title={intl.formatMessage(messages.react)}
disabled={!canReact}
icon='plus'

View file

@ -189,7 +189,7 @@ class ActionBar extends React.PureComponent {
navigator.clipboard.writeText(url);
};
nop = () => {} // hack for reaction add button
handleNoOp = () => {} // hack for reaction add button
render () {
const { status, relationship, intl } = this.props;
@ -272,7 +272,7 @@ class ActionBar extends React.PureComponent {
const reactButton = (
<IconButton
className='plus-icon'
onClick={this.nop} // EmojiPickerDropdown handles that
onClick={this.handleNoOp} // EmojiPickerDropdown handles that
title={intl.formatMessage(messages.react)}
disabled={!canReact}
icon='plus'