Page MenuHomePhabricator

As a reader on Wikipedia, I'd like to be able to easily thank editors for their contributions from the Article as a Living Document model
Closed, ResolvedPublic

Description

User story

As a reader on Wikipedia, I'd like to be able to easily thank editors for their contributions from the Article as a Living Document model

Design details

This flow should follow closely the flow for thanking editors in the history view (on individual revisions). A prototype shows the basic features here: https://wikimedia.invisionapp.com/overview/History-and-Diffs-Testing---Wikimania-ck39lyx0o2ty8010jxbvysnv8/screens

Designs

Logged in
No action takenUser education (on first use of feature)ConfirmationRe-tapping
1 - Thank flow - Article as a living document.png (1×750 px, 145 KB)
2 (user edu) - Thank flow - Article as a living document.png (1×750 px, 118 KB)
3 (confirm) - Thank flow - Article as a living document.png (1×750 px, 143 KB)
4 (second tap) - Thank flow - Article as a living document.png (1×750 px, 148 KB)
https://zpl.io/bAM4pQehttps://zpl.io/adOP5Gnhttps://zpl.io/V4WvxM4https://zpl.io/a8MP44E
Logged out
No action takenUser education (on tap)
1a (logged out) - Thank flow - Article as a living document.png (1×750 px, 146 KB)
2a (logged out) - Thank flow - Article as a living document.png (1×750 px, 114 KB)
https://zpl.io/awKmekjhttps://zpl.io/agArdJD

Event Timeline

  • Move tappedThank and associated unctions from DiffContainerViewController to a protocol and protocol extension.
  • Test that I didn't break anything in DiffContainerViewController.
  • Make Article as a Living Doc conform to new protocol.

This is looking really good @MattClienman! Some small design bugs

  • On the 'Publicly send thanks' pop-up would it be possible to remove the blue boarder around the button? -- realizing this might be a bigger ask as it seems to be on all buttons of this style, so I can also file a separate ticket if that's better.
  • On an iPhoneX the success and other toasts are floating a bit above the bottom of the screen. Could we extend the fill of the toast to the bottom of the screen to avoid this little gap?
JMinor claimed this task.
JMinor subscribed.

Thanks!