Prefer null to falsy, use correct plural
This commit is contained in:
parent
1a4df38500
commit
f71ac93946
1 changed files with 3 additions and 2 deletions
|
@ -103,7 +103,7 @@ module.exports = React.createClass({
|
||||||
</span>
|
</span>
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
let leaveSummary = '';
|
let leaveSummary = null;
|
||||||
if (leavers) {
|
if (leavers) {
|
||||||
leaveSummary = (
|
leaveSummary = (
|
||||||
<span>
|
<span>
|
||||||
|
@ -186,7 +186,8 @@ module.exports = React.createClass({
|
||||||
toggleButton = (
|
toggleButton = (
|
||||||
<a onClick={this._toggleSummary}>{expanded?'collapse':'expand'}</a>
|
<a onClick={this._toggleSummary}>{expanded?'collapse':'expand'}</a>
|
||||||
);
|
);
|
||||||
let noun = (joinAndLeft === 1 ? 'user' : 'others');
|
let plural = (joinEvents.length + leaveEvents.length > 0) ? 'others' : 'users';
|
||||||
|
let noun = (joinAndLeft === 1 ? 'user' : plural);
|
||||||
|
|
||||||
summaryContainer = (
|
summaryContainer = (
|
||||||
<div className="mx_EventTile_line">
|
<div className="mx_EventTile_line">
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue