OpenCloud
/
riot-web
mirror of https://github.com/vector-im/riot-web
1
0
Fork 0
Code Issues Releases Wiki Activity

Use less confusing values on radio buttons

pull/21833/head
Luke Barnard 2018-03-29 17:25:06 +01:00
parent 18ec690e2e
commit 2a6d7e6e8e
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/components/structures/GroupView.js
View File

@ -400,6 +400,9 @@ FeaturedRoom.contextTypes = GroupContext;
RoleUserList.contextTypes = GroupContext; RoleUserList.contextTypes = GroupContext;
FeaturedUser.contextTypes = GroupContext; FeaturedUser.contextTypes = GroupContext;
const GROUP_JOINABLE = "GROUP_JOINABLE";
const GROUP_NOT_JOINABLE = "GROUP_NOT_JOINABLE";
export default React.createClass({ export default React.createClass({
displayName: 'GroupView', displayName: 'GroupView',
@ -613,7 +616,7 @@ export default React.createClass({
_onJoinableChange: function(ev) { _onJoinableChange: function(ev) {
this.setState({ this.setState({
joinableForm: { isJoinable: ev.target.value === "true" }, joinableForm: { isJoinable: ev.target.value === GROUP_JOINABLE },
}); });
}, },
@ -950,7 +953,7 @@ export default React.createClass({
<div> <div>
<label> <label>
<input type="radio" <input type="radio"
value="false" value={GROUP_NOT_JOINABLE}
checked={!this.state.joinableForm.isJoinable} checked={!this.state.joinableForm.isJoinable}
onClick={this._onJoinableChange} onClick={this._onJoinableChange}
/> />
@ -962,7 +965,7 @@ export default React.createClass({
<div> <div>
<label> <label>
<input type="radio" <input type="radio"
value="true" value={GROUP_JOINABLE}
checked={this.state.joinableForm.isJoinable} checked={this.state.joinableForm.isJoinable}
onClick={this._onJoinableChange} onClick={this._onJoinableChange}
/> />