Stop setting IS input field on account change

This stops setting a value in the IS input on account change. While it may have
been marginally useful if you have the form open and change on a different
device, it also seems to pick up changes on the current device, leading to
strange UX locally.

Fixes https://github.com/vector-im/riot-web/issues/10756
Fixes https://github.com/vector-im/riot-web/issues/10757
pull/21833/head
J. Ryan Stinnett 2019-09-06 14:27:20 +01:00
parent 46ee52a406
commit 5fddb20d86
1 changed files with 2 additions and 9 deletions

View File

@ -93,18 +93,11 @@ export default class SetIdServer extends React.Component {
onAction = (payload) => { onAction = (payload) => {
// We react to changes in the ID server in the event the user is staring at this form // We react to changes in the ID server in the event the user is staring at this form
// when changing their identity server on another device. If the user is trying to change // when changing their identity server on another device.
// it in two places, we'll end up stomping all over their input, but at that point we
// should question our UX which led to them doing that.
if (payload.action !== "id_server_changed") return; if (payload.action !== "id_server_changed") return;
const fullUrl = MatrixClientPeg.get().getIdentityServerUrl();
let abbr = '';
if (fullUrl) abbr = abbreviateUrl(fullUrl);
this.setState({ this.setState({
currentClientIdServer: fullUrl, currentClientIdServer: MatrixClientPeg.get().getIdentityServerUrl(),
idServer: abbr,
}); });
}; };