So I was playing around with head during the sprint this weekend and noticed the following funky behavior:

  1. some, not all, of the vertical tabs items show up as 'fields' at admin/build/node-type/article/fields
  2. moving them around in the 'manage fields' table does not change their order in the vertical tabs themselves
  3. you get some funky behavior with the other fields in the 'manage fields' table

Manage Fields' tab screencap:

Vertical Tabs screencap:

Obviously , this is incorrect behavior and a usability issue-- but I'm not sure what the fix is. I would guess removing anything that's a vertical tab from the 'manage fields' table would be probably be the easiest. However, in that case, should the entire vertical tabs element be added to the 'manage fields' table so as to be able to be positioned on the node/edit form as a unit?

A another option would be to have all the elements in the vertical tabs appear in the 'manage fields' table, but then there needs to be some logic to keep them grouped together, reorderable within themselves, and reorderable as a group. Much more complicated.

CommentFileSizeAuthor
vt_node_edit.png73.82 KBWorldFallz
vt_manage_fields.png49.53 KBWorldFallz

Comments

yched’s picture

Project: Drupal core » Content Construction Kit (CCK)
Version: 7.x-dev » 7.x-2.x-dev
Component: field system » General

This is not for core, this is for CCK. The 'reordering of non-field items' feature needs to be amended after the vertical-tabs reorganisation of the node form.

Not exactly sure of the best way to do this. At least, we should remove 'file attachments' and 'menu settings' from the list of reorderable pseudo fields.

karens’s picture

Possibly related to #409144: content_extra_weights gets corrupted and breaks the ordering of form elements on node edit form, problems getting the D6 version of Vertical Tabs working.

Plus the node form for D7 is changing again, so not sure how this is affected.

dave reid’s picture

Issue tags: +vertical tabs

Adding tag

karens’s picture

Status: Active » Fixed

The Manage Fields UI is now in core and all the code around this has changed. Assuming it doesn't apply any more. If there is still a problem, we need a new issue that relates to the new way the code is designed.

Status: Fixed » Closed (fixed)
Issue tags: -vertical tabs

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