Page MenuHomePhabricator

Anon Edit warning screen has UI bugs [design debt]
Closed, ResolvedPublic1 Estimated Story Points

Description

Anonymous editing warning screen has UI bugs

  • Message is sticking the edges of the screen
  • buttons sticking together

Proposal
- Improve the message to reflect the warning in a better way.
- use "warning" icon which has better recall

  • Adding correct paddings around message and buttons
  • Include benefit of logging in or creating account near the call to actions.

Mock
Before

anon-editbefore.png (1×750 px, 144 KB)

After

anon-edit.png (1×750 px, 146 KB)

Zeplin spec https://zpl.io/Z1eCptk

Event Timeline

ovasileva triaged this task as Medium priority.May 8 2017, 9:10 AM
Jdlrobson added a project: UI-Standardization.
Jdlrobson added a subscriber: Volker_E.

On top of Baha's question above, another look shows that the editor is using the warningbox class.
@Volker_E I'd argue we should style all warningboxes consistently across desktop/mobile - can we change these everywhere.
Also why are some warning boxes yellow and others gray?

Nirzar changed the task status from Stalled to Open.Jun 29 2017, 10:03 PM
Nirzar updated the task description. (Show Details)
Nirzar moved this task from Incoming to Triaged but Future on the Web-Team-Backlog board.

Since this is blocked on T166915 (in tracking), do we want to remove it from upcoming?

It is not blocked on T166915 as I had de-scoped it for the same reason. this is about the paddings between the message and buttons.

Change 374408 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Add spacing to anonymous editing message

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

Change 374412 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Show CTA at bottom of anonymous editor warning

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

Change 374408 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Add spacing to anonymous editing message

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

Change 374412 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Show CTA at bottom of anonymous editor warning

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