Page MenuHomePhabricator

Adjust the loading indicator to affect only the list of items in the Notification Page
Closed, ResolvedPublic

Description

When switching the filters (i.e., all, read, unread buttons), a big area becomes greyed-out:

Screen Shot 2016-05-30 at 12.56.37.png (1×2 px, 521 KB)

Making the loading pattern to affect only the list (and not the controls on top) would be more expected since that represents the elements being loaded.

As a result, the controls would change their status immediately improving the perceived performance.

Event Timeline

Change 291852 had a related patch set uploaded (by Mooeypoo):
Make the notifications list the pending element

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

Change 291852 merged by jenkins-bot:
Make the notifications list the pending element

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

Checked in betalabs - the loading indicator affects only the area that will be change, not the controls.

Screen Shot 2016-06-01 at 2.09.19 PM.png (638×1 px, 257 KB)

load_indicator.gif (410×656 px, 1 MB)