Consolidate disabling of history options
Signed-off-by: Robin Townsend <robin@robin.town>pull/21833/head
parent
d568e92e06
commit
0117d513ea
|
@ -350,17 +350,14 @@ export default class SecurityRoomSettingsTab extends React.Component<IProps, ISt
|
||||||
const options = [
|
const options = [
|
||||||
{
|
{
|
||||||
value: HistoryVisibility.Shared,
|
value: HistoryVisibility.Shared,
|
||||||
disabled: !canChangeHistory,
|
|
||||||
label: _t('Members only (since the point in time of selecting this option)'),
|
label: _t('Members only (since the point in time of selecting this option)'),
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
value: HistoryVisibility.Invited,
|
value: HistoryVisibility.Invited,
|
||||||
disabled: !canChangeHistory,
|
|
||||||
label: _t('Members only (since they were invited)'),
|
label: _t('Members only (since they were invited)'),
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
value: HistoryVisibility.Joined,
|
value: HistoryVisibility.Joined,
|
||||||
disabled: !canChangeHistory,
|
|
||||||
label: _t('Members only (since they joined)'),
|
label: _t('Members only (since they joined)'),
|
||||||
},
|
},
|
||||||
];
|
];
|
||||||
|
@ -369,7 +366,6 @@ export default class SecurityRoomSettingsTab extends React.Component<IProps, ISt
|
||||||
if (!this.state.encrypted || history === HistoryVisibility.WorldReadable) {
|
if (!this.state.encrypted || history === HistoryVisibility.WorldReadable) {
|
||||||
options.unshift({
|
options.unshift({
|
||||||
value: HistoryVisibility.WorldReadable,
|
value: HistoryVisibility.WorldReadable,
|
||||||
disabled: !canChangeHistory,
|
|
||||||
label: _t("Anyone"),
|
label: _t("Anyone"),
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
@ -384,6 +380,7 @@ export default class SecurityRoomSettingsTab extends React.Component<IProps, ISt
|
||||||
name="historyVis"
|
name="historyVis"
|
||||||
value={history}
|
value={history}
|
||||||
onChange={this.onHistoryRadioToggle}
|
onChange={this.onHistoryRadioToggle}
|
||||||
|
disabled={!canChangeHistory}
|
||||||
definitions={options}
|
definitions={options}
|
||||||
/>
|
/>
|
||||||
</div>
|
</div>
|
||||||
|
|
Loading…
Reference in New Issue