Tasks:

  • Use vertical tabs for reusable and revision fields
  • Ensure that the translation tab when entity_translation is enabled uses vertical tab as well
  • Add useful vertical tab summaries
  • Switch from using CTools dependencies to the D7 states system.
CommentFileSizeAuthor
#5 1911176-cleanup-edit-form.patch8.03 KBDave Reid
#1 1911176-cleanup-edit-form.patch6.22 KBDave Reid
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
FileSize
1911176-cleanup-edit-form.patch6.22 KB

Screenshots of some states:

Lots of things filled out

Collapsed/empty items:

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Active » Needs review
gooddesignusa’s picture

gooddesignusa CreditAttribution: gooddesignusa commented

I just applied this patch and it seems to be working for me. +1
Thank you.

edit: So I noticed in my logs a 404 for fieldable_panels_panes.vertical-tabs.js
I looked in the patch above and it is being referenced but was never part of the patch. It seems to be working fine without it. Also noticed my title link field is render below the body field instead of below the title field.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Issue summary: View changes
Status: Needs review » Needs work

Need that missing file, or a pseudo-code explanation of what it contained so it can be recreated.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs work » Needs review
FileSize
1911176-cleanup-edit-form.patch8.03 KB

Here's the version with the missing file.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs review » Reviewed & tested by the community

Looks good, thanks Dave!

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Parent issue: » #2244541: [meta] Plan for Fieldable Panels Pane 7.x-1.6 release
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

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