Hygiene: Ajax loader on notifications icon should be a real Icon
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Jdlrobson
	Nov 11 2014, 1:15 AM

Description

In notifications.js an Icon is created which has no name. An additionalClass loading is what renders the icon. Introduced in patch https://gerrit.wikimedia.org/r/#/c/172425/

We should create mw-ui-icon-ajax (or icon-ajax for stable/beta)
On a side note: Icon class should throw an error if no name parameter is given.

Version: unspecified
Severity: normal

Details

Reference: bz73251

Event Timeline

• bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:48 AM

• bzimport added a project: Don't use – MobileFrontend-Hygiene.

• bzimport set Reference to bz73251.

Jdlrobson created this task.Nov 11 2014, 1:15 AM

bingle-admin wrote:

Prioritization and scheduling of this bug is tracked on Trello card https://trello.com/c/edxeE6B4

This should be fixed with https://gerrit.wikimedia.org/r/172743/

Jdforrester-WMF edited projects, added Technical-Debt, MobileFrontend-General-or-Unknown; removed Don't use – MobileFrontend-Hygiene.Dec 8 2014, 9:42 PM

Hygiene: Ajax loader on notifications icon should be a real IconClosed, ResolvedPublicActions

Description

Details

Event Timeline

Hygiene: Ajax loader on notifications icon should be a real Icon
Closed, ResolvedPublic
Actions