Page MenuHomePhabricator

Use standardised ajax loader and refresh icon in ConfirmEdit
Open, LowPublic1 Story Points

Description

The fancycaptcha-reload-loading class adds an ajax spinner and .fancycaptcha-reload class to add a reload icon.
The icon is inconsistent with the mobile Minerva skin.

Can these use OOjs UI icons instead or at the very least make use of some standard mediawiki UI icon?

Event Timeline

Jdlrobson created this task.Apr 4 2017, 8:59 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 4 2017, 8:59 PM
Jdlrobson renamed this task from Use standardised ajax loader and refresh icon to Use standardised ajax loader and refresh icon in ConfirmEdit.Apr 4 2017, 9:03 PM
Jdlrobson added a project: VisualEditor.
Jdlrobson moved this task from Incoming to 2014-15 Q4 on the Readers-Web-Backlog board.

What images do you propose? (Sorry, probably I just haven't seen refresh images in OOJs UI?)

I could also imagine, that an intermediate progressbar could be used for a refresh indication, instead of a loading circle?

In VE/NWE we use dialog pending state rather than a rotating/whatever icon, for consistency (and don't load fancycaptcha-reload-loading at all). Once we switch mobile front-end over to this it'll be solved, but ahead of that we could migrate that code to do an indeterminate progress bar if it's urgent.

Jdforrester-WMF triaged this task as Low priority.Apr 10 2017, 7:28 PM
Jdforrester-WMF moved this task from To Triage to Epics on the VisualEditor board.
Jdforrester-WMF set the point value for this task to 1.
Jdlrobson moved this task from Backlog to Later on the Readers-Web-Backlog (Tracking) board.
Jdlrobson moved this task from Needs triage to Triaged on the Mobile board.Jul 19 2017, 1:11 AM

We've recently added the 'reload' icon to OOjs UI (see https://doc.wikimedia.org/oojs-ui/master/demos/?page=icons ) for a different purpose, which might be used here.

Deskana moved this task from Epics to Freezer on the VisualEditor board.Aug 22 2018, 11:42 AM