From 661c1bb3602ba511f7d25b0990d13e691e07125a Mon Sep 17 00:00:00 2001 From: fef Date: Wed, 30 Nov 2022 09:06:14 +0000 Subject: [PATCH] limit number of reactions displayed Too many reactions on a single post quickly get spammy, so they are now sorted by count and only the first MAX_REACTIONS number of different emojis are actually displayed. --- .../flavours/glitch/components/status_reactions.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/app/javascript/flavours/glitch/components/status_reactions.js b/app/javascript/flavours/glitch/components/status_reactions.js index c0ad827fc..cc85a0189 100644 --- a/app/javascript/flavours/glitch/components/status_reactions.js +++ b/app/javascript/flavours/glitch/components/status_reactions.js @@ -1,7 +1,7 @@ import ImmutablePureComponent from 'react-immutable-pure-component'; import PropTypes from 'prop-types'; import ImmutablePropTypes from 'react-immutable-proptypes'; -import { reduceMotion } from '../initial_state'; +import { maxReactions, reduceMotion } from '../initial_state'; import spring from 'react-motion/lib/spring'; import TransitionMotion from 'react-motion/lib/TransitionMotion'; import classNames from 'classnames'; @@ -36,7 +36,10 @@ export default class StatusReactions extends ImmutablePureComponent { render() { const { reactions } = this.props; - const visibleReactions = reactions.filter(x => x.get('count') > 0); + const visibleReactions = reactions + .filter(x => x.get('count') > 0) + .sort((a, b) => b.get('count') - a.get('count')) + .filter((_, i) => i < maxReactions); const styles = visibleReactions.map(reaction => ({ key: reaction.get('name'),