New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 8983 link

Starred by 3 users

Issue metadata

Status: Released
Owner:
Closed: Jun 1
Cc:
Components:
ReleasedIn: 511.0



Sign in to add a comment

Lazy-load gr-avatar images

Reported by andyb...@golang.org, May 10 2018

Issue description

Images shouldn't be loaded until they need to be shown (say if they're below the fold). Additionally, the page waits for the gr-avatar images to be loaded before the onload event is fired. The images loaded only have a cache-control max-age of one day.

Given the number of avatar images (should the instance be configured to show them) on each page load, this could be an easily-implemented solution with a win on the page Load metric:

+ Don't block the page load event on gr-avatar images.
+ Lazy-load based on visibility.
 
Project Member

Comment 1 by logan@google.com, May 14 2018

Labels: Performance Triaged-Yes Priority-2
Owner: vikt...@google.com
I wonder if this has something to do with  issue 8400 .
Project Member

Comment 2 by wyatta@google.com, Jun 1

Status: Submitted (was: New)
https://gerrit-review.googlesource.com/c/gerrit/+/179450
Project Member

Comment 3 by wyatta@google.com, Jun 4

ReleasedIn: 511.0
Project Member

Comment 4 by wyatta@google.com, Jun 4

Status: Released (was: Submitted)
Labels: FixedIn-2.16

Sign in to add a comment