You are now unable to manage your fields directly from the content types page - we need to add this to the operations table.

Quicksketch suggested we should also have the Display Fields.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yched’s picture

Component: field system » node system

That's not a field issue anymore, but for fieldable types.
Also, we'll need similar links on the 'vocabularies overview table' at admin/structure/taxonomy

jhedstrom’s picture

Status: Active » Needs review
FileSize
972 bytes

The attached patch adds the manage fields and manage display links to the content type admin screen.

Status: Needs review » Needs work

The last submitted patch failed testing.

jhedstrom’s picture

Status: Needs work » Needs review
FileSize
1.12 KB

Ah, I guess the test bot can't apply git patches. Here's a cvs diff.

yched’s picture

Status: Needs review » Needs work

Thanks for tackling this !
- I'd suggest moving 'delete' at the end, though.
- we need to check whether field_ui.module is enabled first. If not, the links go to a 404.

+ a screenshot would probably help this being reviewed and committed ?

Bojhan’s picture

Agreed with yched, can't apply patches so waiting for a screenshot in order to set this RTBC

jhedstrom’s picture

Status: Needs work » Needs review
FileSize
36.79 KB
1.38 KB

This patch does the 2 things yched suggested (delete to the end, and checks for the field_ui module). Also, attached is a screenshot.

yched’s picture

Code-wise, this works for me. I'll let Bojhan validate.

moshe weitzman’s picture

looks good to me ... test bot can apply git patches if you use the --no-prefix option.

Bojhan’s picture

I can confirm it works, any idea where all the spacing comes from though? Look at the spacing between edit and manage field and between manage field and display fields.

jhedstrom’s picture

Regarding column width, it appears to be the nature of tables that are pushed to 100% width. Attached is a screenshot from FF3.5 with the 100% table width disabled. Without specifically targeting this table/column in the theme, I'm not sure if there is a general fix.

Bojhan’s picture

Status: Needs review » Reviewed & tested by the community

OK, RTBC - we can do styling cleanups later more consistently.

sun’s picture

Interestingly, the same output could be achieved with contextual links. ;) Just want to mention possibilities for D8.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. We can continue to discuss 'contextual links' if desired.

yched’s picture

jhedstrom: Thanks ! Care to do the same with admin/structure/taxonomy/list ?

sun: I do think a link to 'manage fields' - or at least 'edit content type' would definitely makes sense in node contextual links.

jhedstrom’s picture

sun’s picture

FileSize
111.45 KB

What I meant is that the output we see here - and perhaps "operations" in general - _are_ contextual links.

manage_fields.links_.png

Status: Fixed » Closed (fixed)
Issue tags: -Usability, -D7UX

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