Closed Bug 985393 Opened 10 years ago Closed 10 years ago

Avoid loading icons the user can't see on homepage

Categories

(Marketplace Graveyard :: Consumer Pages, defect, P2)

Product:
Marketplace Graveyard
Component:
Consumer Pages
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED DUPLICATE of bug 898320

People

(Reporter: mat, Assigned: mat)

References

Details

(Keywords: perf) 

When viewing the homepage on a small screen, only 3 featured apps are shown, you need to follow the "View All" link to see them all. However, looking at the network tab I see that we load the icons for all featured apps and not just those 3.

We should do something similar than we did in bug 983346 to avoid loading those icons when they are hidden.
Actually, it looks like this also happens on a wide screen, if there are more than 7 apps.
Summary: Avoid loading icons the user can't see on mobile homepage → Avoid loading icons the user can't see on homepage
Assignee: nobody → mpillard
https://github.com/mozilla/fireplace/pull/405
Status: NEW → ASSIGNED
Yeah, that's certainly a better approach. If your pull request is going in prod this week then we can forget about mine, otherwise this might be good to have in the meantime.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.