hide lazy loading behind feature flag
This commit is contained in:
parent
866b4bb067
commit
af07d73432
3 changed files with 13 additions and 2 deletions
|
@ -107,7 +107,9 @@ class MatrixClientPeg {
|
||||||
// the react sdk doesn't work without this, so don't allow
|
// the react sdk doesn't work without this, so don't allow
|
||||||
opts.pendingEventOrdering = "detached";
|
opts.pendingEventOrdering = "detached";
|
||||||
|
|
||||||
opts.filter = await this.matrixClient.createFilter(FILTER_CONTENT);
|
if (SettingsStore.isFeatureEnabled('feature_lazyloading')) {
|
||||||
|
opts.filter = await this.matrixClient.createFilter(FILTER_CONTENT);
|
||||||
|
}
|
||||||
|
|
||||||
try {
|
try {
|
||||||
const promise = this.matrixClient.store.startup();
|
const promise = this.matrixClient.store.startup();
|
||||||
|
|
|
@ -314,7 +314,11 @@ module.exports = React.createClass({
|
||||||
this.setState({isPeeking: false});
|
this.setState({isPeeking: false});
|
||||||
|
|
||||||
//viewing a previously joined room, try to lazy load members
|
//viewing a previously joined room, try to lazy load members
|
||||||
MatrixClientPeg.get().loadRoomMembersIfNeeded(room.roomId);
|
|
||||||
|
// lazy load members if enabled
|
||||||
|
if (SettingsStore.isFeatureEnabled('feature_lazyloading')) {
|
||||||
|
MatrixClientPeg.get().loadRoomMembersIfNeeded(room.roomId);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
|
|
|
@ -83,6 +83,11 @@ export const SETTINGS = {
|
||||||
supportedLevels: LEVELS_FEATURE,
|
supportedLevels: LEVELS_FEATURE,
|
||||||
default: false,
|
default: false,
|
||||||
},
|
},
|
||||||
|
"feature_lazyloading": {
|
||||||
|
isFeature: true,
|
||||||
|
displayName: _td("Increase performance by loading room members on first view"),
|
||||||
|
supportedLevels: LEVELS_FEATURE,
|
||||||
|
},
|
||||||
"MessageComposerInput.dontSuggestEmoji": {
|
"MessageComposerInput.dontSuggestEmoji": {
|
||||||
supportedLevels: LEVELS_ACCOUNT_SETTINGS,
|
supportedLevels: LEVELS_ACCOUNT_SETTINGS,
|
||||||
displayName: _td('Disable Emoji suggestions while typing'),
|
displayName: _td('Disable Emoji suggestions while typing'),
|
||||||
|
|
Loading…
Reference in a new issue