Next, the "flow" of things is pretty jarring. Upon submitting a new field I'm taken to an intermediate page for field settings:

Field Settings

Sometimes said page says "there's nothing to configure here". Well then DON'T show me a page! ;) When I click "Save field settings" I'd expect to get taken back to where I was, like all other settings pages in Drupal. Instead, I'm taken to another settings form, this one much longer, but which includes the settings I configured a second ago! Very silly, and very confusing.

Once the field is created, I find that it exposes not one, not two, but THREE settings forms, two of them with "mystery meat" links:

Settings Galore!

http://localhost/core/admin/structure/node-type/page/fields/field_new_fi... Settings for the field type itself only, related to default values, data storage size, etc.

http://localhost/core/admin/structure/node-type/page/fields/field_new_fi... A widget selection only.

http://localhost/core/admin/structure/node-type/page/fields/field_new_fi... That long form again, which has a duplicate of the settings form at field-settings.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jolos’s picture

In the 'manage field' form (last screenshot), it seems a bit unnessecary to have a different form for adjusting the widget type.
Maybe it's better to provide a select list here ( instead of a link to a separate form ), so you can change the widget type immediately on the 'manage fields' form?

Then you're down to 2 settings forms, and I think having 2 different forms wouldn't hurt. With cck I often had to look twice while discerning between node-type specific settings and field specific settings. Offering 2 different forms would certainly help separating these two.
For now, only the field-settings form has a separate form. If you want to change the node-type settings you have to visit 'that long form' ( /admin/structure/node-type/page/fields/fields_new_field ). But you can edit the field specific settings there too, and that's , as webchick mentions, confusing. I think removing the field specific part there is the most obvious solution.

So when creating a new field the workflow is like this:

manage fields > add new field > field specific settings > node-type specific settings > manage fields

changing settings:

manage fields > field/node-type specific settings > manage fields

Indeed, clicking 'save field settings' doesn't take you to the same page. However, I do find these flows quite intuitive.
The first workflow (creating a new field) guides the user through the process of creating a field. If clicking 'save field settings' kept you on the same page this wouldn't be possible.

In short my suggestions:

  • integrate the widget form into the manage fields form
  • keep field and node-type specific as much separated as possible
  • remove the general field settings from the /fields/fields_new_field form

Some other remarks concerning the manage fields form:

(1) I think it isn't very clear what the 'Field' column does.
It should be clear that clicking "boolean" takes you to a form where you can change
the general settings. Idem for 'edit'.

(2) the field settings form gives me the following helptext:

These settings apply to the new_field field everywhere it is used. These settings impact the way that data is stored in the database and cannot be changed once data has been created.

Ok, I added some data ( content? ), and indeed I can't change it (I get a warning). But when I go to that long form, there doesn't seem to a problem?

(3) the field-settings form doesn't give me the option to store multiple values? ( I think it should )

that's it for now

yched’s picture

Component: field system » field_ui.module
jolos’s picture

Status: Active » Needs review
FileSize
32.8 KB
6.06 KB

My very first patch, yeeha! :-)

This patch doesn't address all the issues I discussed in my previous post.

In short I separated the field-settings form and the instance (node-type specific ) form.
I als added a select list for the widget type on the manage fields form. See the attached screenshot for the result.
( Not very surprising though )

Tell me what you think!

Status: Needs review » Needs work

The last submitted patch failed testing.

sun.core’s picture

Priority: Critical » Normal
Status: Needs work » Postponed (maintainer needs more info)

This is no longer critical. Actually, is this is a hidden duplicate of other issues in the queue?

webchick’s picture

Status: Postponed (maintainer needs more info) » Closed (duplicate)