mirror of
https://github.com/element-hq/element-web
synced 2024-11-24 10:15:43 +03:00
Fix unnecessary blurhash rendering
This commit is contained in:
parent
b8f6d2926c
commit
40cb8e8fc6
1 changed files with 14 additions and 2 deletions
|
@ -55,6 +55,7 @@ export default class MImageBody extends React.Component<IBodyProps, IState> {
|
||||||
static contextType = MatrixClientContext;
|
static contextType = MatrixClientContext;
|
||||||
private unmounted = true;
|
private unmounted = true;
|
||||||
private image = createRef<HTMLImageElement>();
|
private image = createRef<HTMLImageElement>();
|
||||||
|
private timeout?: number;
|
||||||
|
|
||||||
constructor(props: IBodyProps) {
|
constructor(props: IBodyProps) {
|
||||||
super(props);
|
super(props);
|
||||||
|
@ -146,12 +147,14 @@ export default class MImageBody extends React.Component<IBodyProps, IState> {
|
||||||
};
|
};
|
||||||
|
|
||||||
private onImageError = (): void => {
|
private onImageError = (): void => {
|
||||||
|
this.clearBlurhashTimeout();
|
||||||
this.setState({
|
this.setState({
|
||||||
imgError: true,
|
imgError: true,
|
||||||
});
|
});
|
||||||
};
|
};
|
||||||
|
|
||||||
private onImageLoad = (): void => {
|
private onImageLoad = (): void => {
|
||||||
|
this.clearBlurhashTimeout();
|
||||||
this.props.onHeightChanged();
|
this.props.onHeightChanged();
|
||||||
|
|
||||||
let loadedImageDimensions;
|
let loadedImageDimensions;
|
||||||
|
@ -267,6 +270,13 @@ export default class MImageBody extends React.Component<IBodyProps, IState> {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private clearBlurhashTimeout() {
|
||||||
|
if (this.timeout) {
|
||||||
|
clearTimeout(this.timeout);
|
||||||
|
this.timeout = undefined;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
componentDidMount() {
|
componentDidMount() {
|
||||||
this.unmounted = false;
|
this.unmounted = false;
|
||||||
this.context.on('sync', this.onClientSync);
|
this.context.on('sync', this.onClientSync);
|
||||||
|
@ -281,8 +291,9 @@ export default class MImageBody extends React.Component<IBodyProps, IState> {
|
||||||
} // else don't download anything because we don't want to display anything.
|
} // else don't download anything because we don't want to display anything.
|
||||||
|
|
||||||
// Add a 150ms timer for blurhash to first appear.
|
// Add a 150ms timer for blurhash to first appear.
|
||||||
if (this.media.isEncrypted) {
|
if (this.props.mxEvent.getContent().info?.[BLURHASH_FIELD]) {
|
||||||
setTimeout(() => {
|
this.clearBlurhashTimeout();
|
||||||
|
this.timeout = setTimeout(() => {
|
||||||
if (!this.state.imgLoaded || !this.state.imgError) {
|
if (!this.state.imgLoaded || !this.state.imgError) {
|
||||||
this.setState({
|
this.setState({
|
||||||
placeholder: 'blurhash',
|
placeholder: 'blurhash',
|
||||||
|
@ -295,6 +306,7 @@ export default class MImageBody extends React.Component<IBodyProps, IState> {
|
||||||
componentWillUnmount() {
|
componentWillUnmount() {
|
||||||
this.unmounted = true;
|
this.unmounted = true;
|
||||||
this.context.removeListener('sync', this.onClientSync);
|
this.context.removeListener('sync', this.onClientSync);
|
||||||
|
this.clearBlurhashTimeout();
|
||||||
}
|
}
|
||||||
|
|
||||||
protected messageContent(
|
protected messageContent(
|
||||||
|
|
Loading…
Reference in a new issue