The new Lazy load image attachments doesn't work

[teamsoo]

Description:

According to #10608, our instance of RC doesn't show the image when we point at it. Is there a way to disable this Lazy load feature?

Server Setup Information:

• Version of Rocket.Chat Server: 0.65.0
• Operating System: Ubuntu 16.04.1 LTS
• Deployment Method(snap/docker/tar/etc): docker (official rocketchat/rocket.chat)
• Number of Running Instances: 1
• DB Replicaset Oplog: Disabled
• Node Version: n/a
• mongoDB Version: n/a

Steps to Reproduce:

1. Upload image

Expected behavior:

Point at the preview image to see the real image

Actual behavior:

Nothing happened when pointing the preview image

Relevant logs:

n/a

[ggazzo]

@teamsoo could you check if you have some error on your console? We will think in a way to disable, but would be useful to fix this problem.

[teamsoo]

@ggazzo Hi, thanks for the response. I'm seeing this error when scrolling pass the preview image. Not sure if it's related.

[teamsoo]

yeah. I believe they're related.

a=e.firstNode.getBoundingClientRect();if(a.top>=-100&&a.left>=0&&a.top<=(window.innerHeight||document.documentElement.clientHeight))return l(e)})};window.addEventListener("resize",window.lazyloadtick);

[ppcat]

got same error after upgrade to 0.65.0

[JSzaszvari]

I saw this issue too. I did a bunch of testing and it seems to work fine up until the point that this PR is merged - #10887

It seems to work OK, but then once you merge that particular PR in, the images will not fully load and stay blurry

@ggazzo FYI

[rocket-cat]

You don't have permission to use this label bug

[JSzaszvari]

@rocket-cat label add "type: bug"

[traxanos]

i can confirm this

[ggazzo]

sorry guys, nrr template I'm working to fix it

[IQ2022]

I have same issue on my site.. please fix - I can't even roll back to 64

[IQ2022]

i have docker build and the above still an issue

[earlng]

Can confirm. I updated from 0.64.2 to 0.65.0, I'm also getting this issue.

[muhv7]

Is there any way to put in the fix while being on snap without waiting for the next candidate release?

[vynmera]

Editing Snaps is usually difficult. You can try these tips: https://askubuntu.com/questions/919091/why-snap-files-cannot-be-modified-in-any-way
Otherwise, you'll have to clone the fixed branch and import your data.

This should really be a hotfix though. Not every team can afford having to switch to a local build...

[IQ2022]

can we confirm on the ETA for next release please???

[alvarow]

I also see this on 0.65.0 since I upgraded it last night.

[muhv7]

I agree on the hotfix part. Can't roll back to 64.2 either, site wont come up with it.

[teamsoo]

My workaround is adding this script to Custom Script for Logged In Users in Layout

setInterval(function() {
	document.querySelectorAll(".attachment-image .lazy-img").forEach(function(el) { 
    	el.src = el.dataset.src;
		el.removeAttribute('data-src');
		el.className = el.className.replace('lazy-img', '');
	})
}, 1000)

Thanks @ggazzo for the script. 👍