Page MenuHomePhabricator

Review Edit Cards v2.0 implementation for desktop
Open, Needs TriagePublic

Description

Task overview

This tasks involves the work with deciding how the Edit Cards v2.0 designs – T225725 – should be reflected in desktop VE. What – if anything – about how Edit Cards are being designed for mobile should be changed on desktop?

Background

Mobile and desktop VE codebases are nearly the same. There are maintenance and testing costs associated with these codebases diverging.

As a result, we should assume all changes on mobile will be reflected on desktop and vice versa unless we see a reason for this not to happen.

"Done"

  • Engineering knows what – if anything – about how context items are treated on mobile need to be changed on desktop

Event Timeline

ppelberg created this task.Jun 20 2019, 9:50 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 20 2019, 9:50 PM
ppelberg updated the task description. (Show Details)Jun 24 2019, 5:55 PM
Trizek-WMF added a subscriber: Trizek-WMF.

Just chatted with @Esanders about this and we think the best way to approach this is to mock up a desktop view of the card (rather than picking out bit by bit).

@Esanders I started to take a stab at this work and I actually think that it would be easiest to discuss areas where it is impacting the desktop experience during implementation.

Capturing a google chat conversation that @Esanders and I am having regarding desktop implementation of the cards.

This is what the design currently looks like:

This is what the design would look like with the styles applied:

From our convo:

I don't think the X is required on desktop

I think the extra vertical space from the large remove link button isn't necessarily an improvement

On desktop the context covers up other parts of the document, and may trigger a scroll if it is too low down, so I think keeping in compact vertically is a good idea

On mobile we gave ourselves roughly the height of the virtual keyboard to play with, and we're concerned about having large buttons for touch. Those things are unique to mobile I think.

For a more accurate view of the current design applied to desktop:

Here's my latest mockup.

In this iteration:

  • got rid of title bar
  • got rid of cancel button
  • applied updated design styles

cc/@Esanders @ppelberg

I'm removing the design tag on this as we have finalized this in terms of what we will ship for V2.

JTannerWMF moved this task from To Triage to Needs Discussion/Analysis on the VisualEditor board.
JTannerWMF added a subscriber: JTannerWMF.

@Esanders please coordinate with @iamjessklein and @ppelberg on the status of this

During Triage, we discussed this task representing the divergence between desktop and mobile but we have now we are shifting focus to Talk Pages so for that reason this task is deprioritized.