Page MenuHomePhabricator

Delay spinner showing for graphs for 1s
Closed, ResolvedPublic

Description

The progress spinner appears even if it's only going to be visible for a split second. The current performance perception best practice is to only have spinner/progress bars display for processes that take a significant amount of time. Otherwise, the spinner brings attention to a process that may actually be fast, giving the false impression that it's going to take a while. Basically, don't draw attention to something that might be fast. The recommended fix is simple: just wait for 1 second (or maybe even more) before displaying the spinner. In a lot of cases, the graph will appear in less time than that short wait and the user won't have to view the spinner at all. It will look just like images loading on the page.

Event Timeline

Change 614916 had a related patch set uploaded (by Seddon; owner: seddon):
[mediawiki/extensions/Graph@master] Remove embed and add spinner delay

https://gerrit.wikimedia.org/r/614916

Seddon removed Seddon as the assignee of this task.Jul 1 2021, 2:41 AM
Seddon subscribed.
herron triaged this task as Medium priority.Jul 1 2021, 5:13 PM

Change 614916 merged by jenkins-bot:

[mediawiki/extensions/Graph@master] Remove embed and add spinner delay

https://gerrit.wikimedia.org/r/614916

jijiki claimed this task.
jijiki removed a project: serviceops.