Closed Bug 867472 Opened 12 years ago Closed 12 years ago

Nuke unnecessary screenshot assets from app listing pages (home/category)

Categories

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

x86
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED
2013-05-09

People

(Reporter: spasovski, Assigned: spasovski)

References

()

Details

(Whiteboard: [fireplace] p=2)

The network load of unused images is insane. Thanks for finding this Basta. 1. Load homepage. 2. Open the network tab in your favorite debugger. 3. Notice all the screenshots being requested and never used. Fix this and watch the subsequent delirious joy.
Assignee: nobody → dspasovski
Whiteboard: [fireplace] p=2
This is P1, IMO. This causes upwards of 1MB of bandwidth on each uncached category hit.
Priority: -- → P1
This is because we're using the listing style on mobile version of the category landing pages/homepage, but we never allow the user to toggle to the graphical view - it's always the list view. This is my bad - we should fix this in our call when we render the `market_title.html` template.
This begs the question though ... I've been doing `display: none` on the screenshot trays. If we're loading the list view by default for mobile and the screenshots are hidden by the default, and evidently we're still making HTTP requests to fetch these images, that's penalizing our mobile users. We might want to lazy load these instead of using straight <img> tags. Hmm ...
Summary: nuke unnecessary screenshot assets from app listing pages (home/category) → Nuke unnecessary screenshot assets from app listing pages (home/category)
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2013-05-09
You need to log in before you can comment on or make changes to this bug.