maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix pluralization in FollowUsers component (#7928) [deploy]

Browse source
This commit is contained in:
Vaidehi Joshi 2020-05-18 11:38:47 -07:00 • committed by GitHub
parent b1de4cb1cf
commit 3b9fe2658e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 11 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
app/javascript/onboarding/components/FollowUsers.jsx
View file

@ -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 (
<button type="button" onClick={() => this.handleSelectAll()}>
{selectedUsers.length !== users.length
? `Select all ${users.length} people`
: 'Deselect all'}
{followText}
</button>
);
}