Arsharth/hydrogen-web
1
0
Fork 0
forked from mystiq/hydrogen-web
Code Issues Pull requests Projects Releases Packages Wiki Activity

override emitChange so no need to clone option object for all tiles

Browse source 
instead, we don't store the emitChange in the options but rather on
the tile itself.
This commit is contained in:
Bruno Windels 2022-05-09 14:09:45 +02:00
parent 139a87de99
commit 6beff7e552
2 changed files with 11 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/domain/session/room/timeline/TilesCollection.js
View file

@ -35,7 +35,7 @@ export class TilesCollection extends BaseObservableList {
_createTile(entry) {
const Tile = this._tileOptions.tileClassForEntry(entry);
if (Tile) {
return new Tile(entry, { ...this._tileOptions });
return new Tile(entry, this._tileOptions);
}
}

13
src/domain/session/room/timeline/tiles/SimpleTile.js
View file

@ -22,6 +22,7 @@ export class SimpleTile extends ViewModel {
constructor(entry, options) {
super(options);
this._entry = entry;
this._emitUpdate = undefined;
}
// view model props for all subclasses
// hmmm, could also do instanceof ... ?
@ -67,16 +68,20 @@ export class SimpleTile extends ViewModel {
// TilesCollection contract below
setUpdateEmit(emitUpdate) {
this.updateOptions({emitChange: paramName => {
this._emitUpdate = emitUpdate;
}
/** overrides the emitChange in ViewModel to also emit the update over the tiles collection */
emitChange(changedProps) {
if (this._emitUpdate) {
// it can happen that after some network call
// we switched away from the room and the response
// comes in, triggering an emitChange in a tile that
// has been disposed already (and hence the change
// callback has been cleared by dispose) We should just ignore this.
if (emitUpdate) {
emitUpdate(this, paramName);
this._emitUpdate(this, changedProps);
}
}});
super.emitChange(changedProps);
}
get upperEntry() {