865813 - App previews should load more gracefully

Status: VERIFIED FIXED

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

Description

[Matt Basta [:basta]]

Currently the behavior is to dump some <img> tags in the preview area and when they show up they show up. This isn't very graceful, and in reality it causes an uncomfortable delay between clicking a link and the perceived page load.

Some options:
- Throbbers
- Placeholder screenshots
- ???

Comment 1

[Maria Sandberg [:mushi]]

Sorry for asking the obvious but what is an app preview?

Comment 2

[Christopher Van Wiemeersch [:cvan]]

(In reply to Maria Sandberg [:mushi] from comment #1)
> Sorry for asking the obvious but what is an app preview?

The screenshots: http://f.cl.ly/items/2j2q2Z16461q38013Z32/Screen%20Shot%202013-04-25%20at%203.35.13%20PM.png

Comment 3

[Christopher Van Wiemeersch [:cvan]]

Michael said let's use throbbers. This is easy.

Comment 4

[Kevin Ngo [:ngoke]]

STR:

1. Search for 'fire', go to expanded/screenshot toggle.

Expected:

Placeholder images are light gray rocketships, images fade in.

Actual:

Alt text is visible before images are loaded or nothing is visible.

Comment 5

[Iulian Timis]

Issue is still reproducing in https://marketplace-dev.allizom.org/ on FF 31 (Win 7) in App details page.
Please see screencast: http://screencast.com/t/MVW8SR3zS

Comment 6

[Kevin Ngo [:ngoke]]

Enabled for app detail pages:

https://github.com/mozilla/fireplace/commit/1852973595475edddf46c678d9fa28e1ac7928ef

Comment 7

[Iulian Timis]

Verified as fixed in https://marketplace-dev.allizom.org/ on FF31 (Win 7).
Postfix screencast http://screencast.com/t/IwRxExwuNFxD
Closing bug.