Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
(not really an overlay.module issue, but the overlay is most affected by this)
Currently we have this:
function node_admin_paths() {
$paths = array(
'node/*/add' => TRUE,
'node/*/edit' => TRUE,
'node/*/delete' => TRUE,
'node/add' => TRUE,
'node/add/*' => TRUE,
);
return $paths;
}
@webchick pointed out that the node revisions page belongs there, for example, since its sort-of administrative-y. Perhaps some others belong there too (book outlines?).
(Note that none of this will have an actual effect on the theme that shows in the overlay unless #615138: Some pages display in the overlay in a non-adminstrative theme goes in also.)
Comment | File | Size | Author |
---|---|---|---|
#3 | node-related-admin-paths-655508-1.patch | 2.01 KB | David_Rothstein |
#1 | node-related-admin-paths-655508-1.patch | 2.01 KB | David_Rothstein |
Comments
Comment #1
David_Rothstein CreditAttribution: David_Rothstein commentedHere's a patch.
After reviewing all node-related paths in core, I decided that book outlines and content translation probably made sense to add here as well. I thought about the statistics module also (node/*/track), but that one seemed like it probably shouldn't make the cut. Obviously there are gray areas here, and contrib modules can refine the list if they want to.
Note that most of these are local tasks, and frankly the experience of clicking on a local task and having an overlay pop up is pretty weird. We should think later about whether many of these local tasks should become local actions instead....
Finally, this patch also removes node/*/add from the list, since as far as I know that path doesn't even exist in core :)
Comment #3
David_Rothstein CreditAttribution: David_Rothstein commentedSomething funny happened there. Let's attach the patch again.
Comment #6
Dave ReidI like it and looks good.
Comment #7
webchickCommitted to HEAD. Thanks!