[go: up one dir, main page]

Page MenuHomePhabricator

Isolate Section Editing: Determine New Workflows
Closed, ResolvedPublic

Description

Collaborate with Product to display and determine how this change will change workflows for users.

Event Timeline

JTannerWMF triaged this task as Medium priority.Nov 20 2018, 5:22 PM
JTannerWMF created this task.

@iamjessklein @Esanders: I have a random thought (which you may already have thought of) inspired by the part of the planning meeting where we talked about how this change will affect desktop VE users. Could we add some functionality in the section edit view allowing the user to easily load the full article for editing? This means they could expand the scope of their edit without having to backtrack. Even better if they can preserve their changes, although I can see why that might be technically difficult.

@Neil_P._Quinn_WMF : With our implementation of section editing (T76541), we're fetching, loading and editing against the full article's data model in any case, so in technical terms a "continue editing with full article" button would be feasible.

I agree with @Neil_P._Quinn_WMF and actually documented this as an issue in the ticket on the prototype itself. T213429 This is an expected affordance and in fact, I thought that's what I would get when I tapped on the first edit pencil.

@Neil_P._Quinn_WMF Yeah I mentioned before some controls to change back to full editing after entering section editing, or load in additional sections one at a time. With the current implementation there would be a few changes necessary, and some UI work too.

@iamjessklein Yes, MobileFrontend has always had the first edit pencil take you to the editor for the lede section, which is different to desktop.

Through user testing, we've identified another edge case which is: if a user wants to edit the section and then quickly move back to editing the whole page.

There are a few potential solutions to this, including some sort of prompt/button on scroll up or down but for now, want to make sure this is documented.

Documented the edge case in a separate ticket: T219812
Closing this issue.