From 3b9fe2658e7bd7d25fae9c1f0b82aa10e20d9b1d Mon Sep 17 00:00:00 2001 From: Vaidehi Joshi Date: Mon, 18 May 2020 11:38:47 -0700 Subject: [PATCH] Fix pluralization in FollowUsers component (#7928) [deploy] --- .../onboarding/components/FollowUsers.jsx | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/app/javascript/onboarding/components/FollowUsers.jsx b/app/javascript/onboarding/components/FollowUsers.jsx index af69db14e..8b4a4d1b3 100644 --- a/app/javascript/onboarding/components/FollowUsers.jsx +++ b/app/javascript/onboarding/components/FollowUsers.jsx @@ -115,15 +115,21 @@ class FollowUsers extends Component { renderFollowToggle() { const { users, selectedUsers } = this.state; - if (users.length === 0) { - return ''; + let followText = ''; + + if (selectedUsers.length !== users.length) { + if (users.length === 1) { + followText = `Select ${users.length} person`; + } else { + followText = `Select all ${users.length} people`; + } + } else { + followText = 'Deselect all'; } return ( ); }