I would like to suggest that we create a sub-menu under the "Create Content" that displays the available node types that a user can create. This would be very useful for anyone using admin menu.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | submenu-1247960-2.patch | 785 bytes | stevector |
| #1 | workbench_submenu_node_types-1247960.patch | 972 bytes | droath |
| Screen shot 2011-08-12 at 2.55.31 PM.png | 15.36 KB | droath |
Comments
Comment #1
droath commentedComment #2
stevectorThanks for the patch droath!
I see the use for these links when using admin_menu module. However, I don't see an upside to replicating the node add form at another path. We already have node_add_list at admin/workbench/create. Those links all go to node/add/[content-type]. So first, I think we should be consistent. Workbench should send users to admin/workbench/create/node/add/[content-type] or node/add/[content-type], but not both.
This part of Workbench was written before I got involved so I can only guess as to the motivation. I'm guessing that the reasoning is that we should reuse existing pages unless we're doing something very different at the new path.
The updated patch uses drupal_goto() to redirect from admin/workbench/create/node/add/[content-type] to node/add/[content-type].
Also, droath, please make patches relative to the module and not your site root so they can cleanly apply with git.
Comment #3
droath commentedPatch #2 works! It totally makes sense to do it this way. I should have caught that beforehand.
I will remember to roll my patches relevant to the module, thanks for the heads up! :)
Comment #4
droath commentedComment #5
stevectorCommitted here: http://drupalcode.org/project/workbench.git/commit/a41c942d3885a1c3329bb...