Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
node system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
18 Aug 2009 at 17:48 UTC
Updated:
3 Jan 2014 at 00:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
yched commentedThat'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
Comment #2
jhedstromThe attached patch adds the manage fields and manage display links to the content type admin screen.
Comment #4
jhedstromAh, I guess the test bot can't apply git patches. Here's a cvs diff.
Comment #5
yched commentedThanks 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 ?
Comment #6
Bojhan commentedAgreed with yched, can't apply patches so waiting for a screenshot in order to set this RTBC
Comment #7
jhedstromThis patch does the 2 things yched suggested (delete to the end, and checks for the field_ui module). Also, attached is a screenshot.
Comment #8
yched commentedCode-wise, this works for me. I'll let Bojhan validate.
Comment #9
moshe weitzman commentedlooks good to me ... test bot can apply git patches if you use the --no-prefix option.
Comment #10
Bojhan commentedI 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.
Comment #11
jhedstromRegarding 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.
Comment #12
Bojhan commentedOK, RTBC - we can do styling cleanups later more consistently.
Comment #13
sunInterestingly, the same output could be achieved with contextual links. ;) Just want to mention possibilities for D8.
Comment #14
dries commentedCommitted to CVS HEAD. We can continue to discuss 'contextual links' if desired.
Comment #15
yched commentedjhedstrom: 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.
Comment #16
jhedstromSure. I added #631038: Add Manage Fields and Manage Display links to the vocab dropbutton to track this.
Comment #17
sunWhat I meant is that the output we see here - and perhaps "operations" in general - _are_ contextual links.