Page MenuHomePhabricator

Make it easier to relocate a Reply Tool you've opened on mobile
Closed, ResolvedPublic

Description

In T319148, we fixed an issue that had been causing the mobile Reply Tool to become "disassociated" from sections that contain 1 comment.

In doing the above, we re-encountered a related usability issue: It can be difficult for people to relocate/return to a comment they've started drafting

This task involves the work for addressing said usability issue.

Story

As someone who has started drafting a comment and needs to scroll the talk page to refer back to something someone said previously such that the comment I started drafting with the Reply Tool is no longer in view, I need an easy way to relocate/return to the comment I started drafting, so that I can finish expressing what I'd started with all of the context I need in mind.

Requirements

Meta

  • Platform(s): Mobile
  • Skin(s): Minerva Neue
  • DiscussionTools state: enabled

UX
Prevent people from collapsing the section they have a Reply Tool open within so that they gain access to the omnipresent Return to Reply button that appears when they scroll the Reply Tool off-screen. See T319148#8308834 [i] .|

Done

  • An Approach is decided upon
  • Requirements are documented
  • Requirements are implemented

i. Note: in implementing this approach, people would NOT be prevented from collapsing/expanding other sections.

Event Timeline

ppelberg added a project: Goal.
ppelberg added subscribers: nayoub, Esanders.

Assigning this over to @nayoub to review the approach @Esanders proposed and created this Patch Demo for:
https://patchdemo.wmflabs.org/wikis/4d9ad50c24/wiki/Talk:DiscussionTools?mobileaction=toggle_view_mobile

Thanks, @ppelberg & @Esanders!
Looks great, my only concern is that when the keyboard is opened then I'm not seeing the "Return to reply" button (since the keyboard is on top of it).
I wonder if we should place the button at the top of the screen then?

Looks great, my only concern is that when the keyboard is opened then I'm not seeing the "Return to reply" button (since the keyboard is on top of it).
I wonder if we should place the button at the top of the screen then?

Per the conversations we had offline as a team yesterday:

  1. We are going to move forward with deploying the user experience this patch demo implements: Prevent people from collapsing the section they have a Reply Tool open within so that they gain access to the omnipresent Return to Reply button. See T319148#8308834.
  2. We are going to file a ticket for the issue @nayoub named in T320753#8334473
    • Note: we are NOT going to address this issue right now for two reasons: A) as @matmarex explained offline and in T218414#5027607, "affixing" the Return to reply button would open up a host of other UX issues and// B) we are assuming that people will intuitively know to tap the keyboard to return to the Reply Tool that has since disappeared from their view

Next up:

  • 1. @Esanders to relate the patches that were required to create this patch demo to this task
  • 2. Editing Engineering to code review said patches

Change 842958 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/DiscussionTools@master] WIP Disable section toggling on mobile when replying

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

Change 842957 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/MobileFrontend@master] WIP Add the ability to temporarily disable header toggling

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

Ed marked the patches as work-in-progress, I'm not sure what work is still to be done.

Jdlrobson subscribed.

I've broken out the web-team portion of this into T328981 so we can estimate it separately. Please feel free to edit the request if I've not captured it correctly.

Change 842957 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/MobileFrontend@master] Add the ability to temporarily disable header toggling

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

Change 842957 merged by jenkins-bot:

[mediawiki/extensions/MobileFrontend@master] Add the ability to temporarily disable header toggling

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

Change 842958 merged by jenkins-bot:

[mediawiki/extensions/DiscussionTools@master] Disable section toggling on mobile when replying

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

matmarex updated the task description. (Show Details)