Page MenuHomePhabricator

[Regression Testing] Verify DiscussionTools' core functionality works on mobile as expected
Closed, ResolvedPublic

Description

In T282638, we are going to offer the Reply Tool, New Discussion Tool, and Topic Subscriptions on mobile, within the Read as wiki page view.

This task involves the work with exhaustively reviewing each of these tools/user experiences to identify what issues and unexpected behavior are present.

Observed issues

This section will continue the list of issues/unexpected behavior #Editing QA identifies in the process of testing the tools listed within the ===Minimum test cases section below.

IssueTicket

Testing environment

We'll test in the following environments using the Beta Cluster...

Operating System (Versions)BrowsersAccount state
Android (12, 11)Chrome, FirefoxLogged in+out
iOS (15, 14)SafariLogged in+out

Minimum test cases

Reply Tool

  1. Visit: https://simple.m.wikipedia.beta.wmflabs.org/wiki/Talk:Cats
  2. Locate a comment to respond to
  3. Click the [ reply ] affordance that appears with the comment you identified in "2."
  4. Make sure you are in the tool's Visual mode
  5. Type some text
  6. Format the text in some way(s) (bold, italicize, etc.)
  7. Insert a link
  8. Insert a ping
  9. Publish the comment you drafted in steps "5." through "8."
  10. Locate the comment you published in step "9." and verify it was published to the page in the way you expected it to be
  11. Repeat steps "5." through "10." in the tool's Source mode
  12. Repeat steps "5." through "10." except switch between Source and Visual modes before publishing the comment you drafted

New Discussion Tool

  1. Visit: https://simple.m.wikipedia.beta.wmflabs.org/wiki/Talk:Cats
  2. Locate the Add discussion affordance
  3. Click the Add discussion affordance to start a new topic for discussion
  4. Make sure you are in the tool's Visual mode
  5. Type some text
  6. Format the text in some way(s) (bold, italicize, etc.)
  7. Insert a link
  8. Insert a ping
  9. Publish the discussion you drafted in steps "5." through "8."
  10. Locate the discussion you published in step "9." and verify it was published to the page in the way you expected it to be
  11. Repeat steps "5." through "10." in the tool's Source mode
  12. Repeat steps "5." through "10." except switch between Source and Visual modes before publishing the discussion you drafted

Special cases

Verify support for ConfirmEdit captchas
Full context in T247513

  1. Make sure you are logged out
  2. Open the New Discussion Tool on the mobile site
  3. Add an external link to the body of the new discussion you are drafting
  4. Click the Add topic button
  5. Verify the captcha interface appears
  6. Complete captcha
  7. Tap Add topic
  8. Notice the topic you drafted was published successfully

Verify blocked users are prevented from opening DiscussionTools
Full context in T247513

  1. Visit a mobile talk page from an account that is blocked from editing.
  2. Notice [ reply ] link are: A) visible and B) appear as they normally do
  3. Click any [ reply ] link on the page
  4. Verify a modal dialog appears with error message similar to what is shown in F34186447 (wording will vary by project)

Verify AbuseFilters are being shown as expected
See T240519

Done

  • Tickets are filed for the issues and unexpected behavior Editing QA observed while conducting regression testing on the mobile Reply Tool, New Discussion Tool, and Topic Subscriptions
  • These tickets are documented in the ==== Observed issues section above

Event Timeline

Tacsipacsi renamed this task from [Regression Testing] Verify the Reply and New Discussion Tools' core functionality works as expected to [Regression Testing] Verify the Reply and New Discussion Tools' core functionality works on mobile as expected.Oct 29 2021, 10:17 AM
Tacsipacsi subscribed.
ppelberg renamed this task from [Regression Testing] Verify the Reply and New Discussion Tools' core functionality works on mobile as expected to [Regression Testing] Verify DiscussionTools' core functionality works on mobile as expected.Nov 5 2021, 12:28 AM

Lowering the priority since we decided not to deploy it before the end of year break.

The first round of regression testing on this is done. The two usability issues identified on Beta cluster were:

  1. "Add Discussion" was opening up the old way to add a new topic by redirecting to this URL: https://en.m.wikipedia.beta.wmflabs.org/wiki/Talk:Cats#/talk/new. Upon pressing Back button it was not clear what has happened as a result of clicking that button/interacting with that affordance. Only scrolling all the way down made the NDT visible/discoverable making the whole experience very unintuitive.
  2. When adding a new topic, it was scrolling down to the end of the page where it got added and it was also highlighting it, but the entire section heading along with the highlight was obscured by the "Read as wiki page" footer.

We realized that these issues are not going to surface at wikis where this will be deployed because:

  1. It will be deployed at wikis which will have the New Discussion Tool enabled as well, so the "Add Discussion" link will definitely be present and clicking on it will not redirect to the URL of adding new topic in the old method.
  2. We will disable the MobileFrontend talk overlay at wikis where we will deploy this. So, that will remove the "read as wiki page" option and the need to interact with it, which will also resolve the issue of it obscuring the heading and the highlight of a newly added section.

@ppelberg: I will be re-assigning this task over to @EAkinloose now to file tasks on any other issues identified and to do a final round of testing after any changes/fixes since I will be working on few other projects/assignments next week.

oh, and the special cases are working as expected too. : )

This looks good to me. I'll go ahead to verify this.