Merge pull request #3068 from matrix-org/jryans/reaction-row-button-live
Fix live updates to reaction row buttonspull/21833/head
commit
97019fbfb5
|
@ -116,6 +116,7 @@ export default class ReactionsRow extends React.PureComponent {
|
||||||
return <ReactionsRowButton
|
return <ReactionsRowButton
|
||||||
key={content}
|
key={content}
|
||||||
content={content}
|
content={content}
|
||||||
|
count={count}
|
||||||
mxEvent={mxEvent}
|
mxEvent={mxEvent}
|
||||||
reactionEvents={events}
|
reactionEvents={events}
|
||||||
myReactionEvent={myReactionEvent}
|
myReactionEvent={myReactionEvent}
|
||||||
|
|
|
@ -27,6 +27,8 @@ export default class ReactionsRowButton extends React.PureComponent {
|
||||||
mxEvent: PropTypes.object.isRequired,
|
mxEvent: PropTypes.object.isRequired,
|
||||||
// The reaction content / key / emoji
|
// The reaction content / key / emoji
|
||||||
content: PropTypes.string.isRequired,
|
content: PropTypes.string.isRequired,
|
||||||
|
// The count of votes for this key
|
||||||
|
count: PropTypes.number.isRequired,
|
||||||
// A Set of Martix reaction events for this key
|
// A Set of Martix reaction events for this key
|
||||||
reactionEvents: PropTypes.object.isRequired,
|
reactionEvents: PropTypes.object.isRequired,
|
||||||
// A possible Matrix event if the current user has voted for this type
|
// A possible Matrix event if the current user has voted for this type
|
||||||
|
@ -77,12 +79,7 @@ export default class ReactionsRowButton extends React.PureComponent {
|
||||||
render() {
|
render() {
|
||||||
const ReactionsRowButtonTooltip =
|
const ReactionsRowButtonTooltip =
|
||||||
sdk.getComponent('messages.ReactionsRowButtonTooltip');
|
sdk.getComponent('messages.ReactionsRowButtonTooltip');
|
||||||
const { content, reactionEvents, myReactionEvent } = this.props;
|
const { content, count, reactionEvents, myReactionEvent } = this.props;
|
||||||
|
|
||||||
const count = reactionEvents.size;
|
|
||||||
if (!count) {
|
|
||||||
return null;
|
|
||||||
}
|
|
||||||
|
|
||||||
const classes = classNames({
|
const classes = classNames({
|
||||||
mx_ReactionsRowButton: true,
|
mx_ReactionsRowButton: true,
|
||||||
|
|
Loading…
Reference in New Issue