Page MenuHomePhabricator

Show a spinner on initial load
Closed, ResolvedPublic

Description

Author: neilk

Description:
Waiting for the tutorial and other page elements to show up can be irritating. Show a spinner to indicate something is happening.


Version: unspecified
Severity: enhancement

Details

Reference
bz28315

Related Objects

StatusSubtypeAssignedTask
DeclinedNone
ResolvedNone
Resolvedkaldari

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:25 PM
bzimport added a project: UploadWizard.
bzimport set Reference to bz28315.

neilk wrote:

When I tried this on commons.prototype the spinner appeared above the UploadWizard and did not go away.

Check your solution again with an empty cache?

neilk wrote:

screenshot of problem on Mac OS X Chrome 10

See attachment.

I believe the problem is twofold:

  1. When debug mode is off (as on prototype) ResourceLoader is going to load the spinner image and the rest of everything at the same time, within the same HTTP call. So if that's delayed, while RL packages something maybe, there's no benefit.
  1. Something else about the way this loads is causing jQuery to get confused about making it visible. Possibly, since the div actually has zero content, it doesn't think it has to hide() it.

Attached:

Screen_shot_2011-04-05_at_8.57.31_PM.png (859×1 px, 157 KB)

neilk wrote:

Also, another thing -- the spinner shouldn't be shown immediately. Otherwise we get a nasty flashing effect sometimes. It should only appear if the loading has taken more than 2 seconds. Easy to do with a timeout.

neilk wrote:

Okay, a bit of thrashing occurred as Kaldari and I overwrote each other's changes/ideas/concepts/whatever, but this appears to be fixed in r85517.