Page MenuHomePhabricator

Talk Page error states, alerts and notices
Closed, ResolvedPublic

Description

Why are we doing this?

Represent all error states, alerts, and notices for article talk pages.

Audience story

As a contributor/reader, I want to be notified if/when an error occurs and I cannot complete my intended action.

Relevant information

Below are the screens for some of the cases. Find more information this Figma file under Slide 15.

Unable to load page with no fallbackUnexpected server errorNo internet connection with fallbackUnable to subscribe to topicUnable to publish comment due to no internet connection
Unable to load page with no fallback.png (812×375 px, 15 KB)
Unexpected server error.png (812×375 px, 33 KB)
No internet connection.png (812×375 px, 73 KB)
Unable to subscribe to topic.png (812×375 px, 72 KB)
No internet connection while publishing a comment.png (812×375 px, 49 KB)

Additional note for QA
We had to change the layout logic across all banners of this kind, so please keep an eye out for any regressions around these banners in other areas of the app.

Event Timeline

JMinor renamed this task from Error states, alerts and notices to Talk Page rrror states, alerts and notices.Jul 18 2022, 6:39 PM
LGoto triaged this task as Medium priority.Jul 18 2022, 6:39 PM
LGoto moved this task from Needs Triage to Product Backlog on the Wikipedia-iOS-App-Backlog board.
Dmantena renamed this task from Talk Page rrror states, alerts and notices to Talk Page error states, alerts and notices.Jul 18 2022, 6:40 PM

Some group thoughts from engineering sync (no need for design input at this point)

  • If subscription succeeds, show subscription state on button immediately. Show success toast or failure toast after subscription call goes through. If it fails, don’t worry about resetting subscription button state, it will change to the correct state the next time the user refreshes their screen.
  • Look into what RMessage library can do for toasts (location, icons, buttons)
  • Can we dedupe the first two error states in the mocks? Maybe just go with the first error state? It’ll take longer to build but better for the user.
  • For no internet connection banner, try to figure that out ahead of time with our ReachabilityNotifier class and display whenever they see each view. Don’t wait for the call to fail - a "No internet connection" banner after typing their response is a bit cruel.

Things remaining to develop:

We haven't picked this up yet, so we still need to do everything here.

  • Error upon first load, upon topic publish, upon reply publish, upon subscribe/unsubscribe
  • No internet connection upon first load, upon topic publish, upon reply publish, upon subscribe/unsubscribe
Mazevedo moved this task from Doing to Needs Code Review on the ios-app-v7.0 board.
Mazevedo subscribed.

@OTichonova in Task sync we discussed that the 'go to settings' link would only be able to open the device settings under 'Wikipedia' which does not give the user the opportunity to turn on Wifi. For this reason, we've proposed removing this link.

@Mazevedo Things are looking good! Would it be possible to add a shadow underneath the toasts and snack bars? The designs have a slight drop shadow, and right now it is not present. If we can't do a drop shadow, could we use a .5 pt top (for snack bars) or bottom (for toast) border using the 'border' color of each theme?

@cmadeo @OTichonova We can look into adding these borders/shadows - would you be okay with it applying to all banners throughout the app? That is, including those that are not a part of the talk page feature.

Tsevener moved this task from Doing to Needs Code Review on the ios-app-v7.0 board.

@OTichonova

@Mazevedo noticed in code review that the Sepia theme displays a white background on all of these toasts. Would you like us to change that while we're in here? Thanks!

@cmadeo @OTichonova We can look into adding these borders/shadows - would you be okay with it applying to all banners throughout the app? That is, including those that are not a part of the talk page feature.

Hi Toni!
Sorry, I missed this earlier. Yes, that would be fine for consistency purposes, is it a lot of work to do this?

@OTichonova

@Mazevedo noticed in code review that the Sepia theme displays a white background on all of these toasts. Would you like us to change that while we're in here? Thanks!

Hi! Yes please, updating it to display the correct background color according to the theme would be great. Nice catch!

Sorry, I missed this earlier. Yes, that would be fine for consistency purposes, is it a lot of work to do this?

Nope! It's already done, feel free to take a look in the latest TestFlight. I'll move this back into Design Review column once we get that Sepia color updated.

@OTichonova I went with our chromeBackground theme color, but let me know if this looks off to you and I'll update it to whatever you need:

IMG_0857.PNG (1×750 px, 181 KB)

IMG_0858.PNG (1×750 px, 225 KB)

IMG_0856.PNG (1×750 px, 179 KB)

IMG_0859.PNG (1×750 px, 230 KB)

IMG_0855.PNG (1×750 px, 179 KB)

IMG_0860.PNG (1×750 px, 246 KB)

IMG_0854.PNG (1×750 px, 181 KB)

IMG_0861.PNG (1×750 px, 245 KB)

Hi!
Small UI change:
Could we increase the size of the icons in the toast to 22pts? And decrease the distance between the icon and the text to 12pts?

Thanks!

@OTichonova adding some screenshots to make review easier

Simulator Screen Shot - iPhone 14 Pro - 2022-11-15 at 15.13.21.png (2×1 px, 295 KB)

simulator_screenshot_0ECE3E23-9A26-4F49-B8E0-DC55A1E90492.png (2×1 px, 297 KB)

simulator_screenshot_A341BE12-3C5D-44FE-96A5-2A0BE3FF3DA4.png (2×1 px, 311 KB)

ABorbaWMF subscribed.

Looks good to me on 7.0.0 (2010)

JMinor claimed this task.