Problem/Motivation

The form layout as a table is great but only works on really wide screens. We want to fix that.

This is a followup of #2876239-7: [Feature] Explore ideas about improving the UX.

Proposed resolution

Have a row with the basic controls: disable and alias, and then offer a link for advanced options (like the enhancers). When clicking on that link a row per advanced option is revealed for that particular field.

A proposed mock is something inspired (or not) in:

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#11 commit-interdiff.txt3.81 KBe0ipso
#8 jsonapi_extras-feature_improve_form_ux_further-2876288-8.patch11.02 KBpedrodacosta
#8 Screenshot from 2018-02-27 17-22-33.png66.91 KBpedrodacosta
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

e0ipso created an issue. See original summary.

e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
CreditAttribution: e0ipso as a volunteer and at Lullabot commented

Some more context. This issue spawns from https://www.drupal.org/node/2876239#comment-12074120.

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
CreditAttribution: clemens.tolboom as a volunteer commented
Issue summary: View changes

Added #2876239-7: [Feature] Explore ideas about improving the UX to summary. Hope that is correct.

shadcn’s picture

shadcn CreditAttribution: shadcn at Chapter Three commented
Assigned: Unassigned » shadcn

@e0ipso @clemens.tolboom Thanks. I'll look into this.

e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
CreditAttribution: e0ipso as a volunteer and at Lullabot commented

@arshadcn I see that this is assigned to you. Are you actively working on this?

shadcn’s picture

shadcn CreditAttribution: shadcn at Chapter Three commented

@e0ipso yes. I've been busy this week with work. But back on jsonapi this weekend :)

shadcn’s picture

shadcn CreditAttribution: shadcn at Chapter Three commented
Assigned: shadcn » Unassigned

@e0ipso unfortunately I couldn't get back to this. I'll be busy with work for next month. Unassigning this in case someone else wants to take a look.

pedrodacosta’s picture

pedrodacosta CreditAttribution: pedrodacosta as a volunteer commented
FileSize
Screenshot from 2018-02-27 17-22-33.png66.91 KB
jsonapi_extras-feature_improve_form_ux_further-2876288-8.patch11.02 KB

I've created a patch based on the mockup from "Proposed resolution" section.

pedrodacosta’s picture

pedrodacosta CreditAttribution: pedrodacosta as a volunteer commented
Status: Active » Needs review

  • e0ipso committed 52b547c on 8.x-2.x authored by pedrodacosta 
    Issue #2876288 by pedrodacosta, e0ipso, arshadcn, clemens.tolboom: [...
e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
CreditAttribution: e0ipso as a volunteer and at Lullabot commented
FileSize
commit-interdiff.txt3.81 KB

I made some minor changes on commit.

e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
CreditAttribution: e0ipso as a volunteer and at Lullabot commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.