Closed (fixed)
Project:
Administration menu
Version:
7.x-3.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
18 Mar 2011 at 10:37 UTC
Updated:
2 Mar 2015 at 11:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
pfrenssenI would even suggest to remove these two menu items altogether.
Comment #2
jason.fisher commentedThis can be done through hook_admin_menu_output_alter(&$content):
Comment #3
pfrenssenThanks, that solves it. Very nice that Admin Menu is fully customizable now!
Comment #4
mstrelan commentedThanks jason.fisher
If you want to move the items under the icon menu they can with this code:
Comment #5
geerlingguy commentedThanks for the code in #2 - very helpful!
Comment #6
tsvenson commentedAgree that they should be taken off the toolbar and moved to the icon menu.
Maybe an option in the settings for letting users opt to move them or leave them where they are would be a good way of conforming with the core structure.
Comment #7
dalinIn addition, the words "Index" and "Tasks" are very vague. Especially for someone new to Drupal seeing these two words as the first ones in the menu is utterly confusing. This is an example of the many times in Drupal that we are too reliant on abstract nouns. In addition to moving these items under the icon menu I recommend renaming them to "Configure by module" and "Administration categories". I don't see the need to clutter the admin UI with settings for where to have them - if someone wants to move them out from under the icon menu they can do so with the techniques listed above.
Comment #8
dalinUgh, these are the names that Drupal Core has defined, it's not related to Admin Menu. Well those aren't going to be changed in D7, so we can change them in Admin Menu, and file a core bug for D8.
Comment #9
sunComment #10
Macronomicus commentedPatch in #8 Is the perfect fix!
Cheers!
Comment #11
itangalo commented+1 for this patch. Really helps my 800x600 presentations.
Comment #12
gratefulsk commentedPatch in #8 works great!
Comment #13
davemybes commentedAdding my name to the list of +1 for the #8 patch.
Comment #14
axe312 commented#8 works perfect with 7.x-3.0-rc1 :)
Comment #15
Courtney.B commentedLoves it! Patch in #8 worked w/ 7.x-3.0-rc1.
Comment #16
jenlamptonPatch in #8 still applies cleanly to 3x-dev as well
Comment #17
sunThese tasks/links were originally simply removed instead of relocated. The code for that is merely outdated.
Was there any particular reason for keeping them around?
Comment #18
dalinI find the index sometimes useful in cases where I've installed an unfamiliar module, and the module's config pages are in a non-obvious spot.
Comment #19
naught101 commentedI agree with dalin. Index is useful. Tasks is not. Perhaps index should be altered back to "Administration by module", or just to "admin index"
Comment #20
jenlamptonI don't find either of them useful, and would love to see them both gone :)
Comment #21
itangalo commentedI agree with jenlampton. But maybe an option on the admin menu settings page to choose for yourself?
Comment #22
pfrenssenWell, if you would need this menu item (or any other for that matter), you can put them in with the hook_admin_menu_output_alter() :)
Comment #23
dalinWhile I find the index mildly useful I'm not in favour of a config option. If someone like myself would like to keep the menu item I'd rather that a hook be used.
Comment #24
sunAlright, different approach - let's move them instead.
This patch makes pretty clear that we need to do something about #821128: Ensure reliable link structure and contents for alter hooks (to not break on customizations), so I elevated that issue.
Comment #25
dalin@sun are you opposed to renaming them to something more understandable like the patch in #8?
Comment #26
sunyeah... I don't think that makes them any more clear - they're sorta in line with the surrounding links, which equally aren't super-specific on their context. So, I'd rather keep and move them as is.
If there's going to be a non-administrative Index and non-administrative Tasks at some point, then, well, yeah, that would be a good reason for renaming them. ;)
Comment #27
jenlamptonThis is great, so much cleaner! :)
Comment #28
sunThanks for reporting, reviewing, and testing! Committed to 7.x-3.x and 8.x-3.x.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.
Comment #29
mstrelan commentedThis is fantastic! It's so strange not having "Flush all caches" as the first thing under that icon though :(
Comment #30
itangalo commentedGreat work. Removing my auto-download of the patch above. :-)
Comment #32
cmstom#2 worked great for me. My issue was that I had extra menu items in the top level that only occurred after upgrade from Drupal 6. I had these items in the top level admin menu: Status report, available updates, recent log messages, field list, recent hits, top access denied errors, top page not found errors, top referrers, top search phrases, views plugins, top pages, top visitors, help.
I obviously needed to clean up all these extra menu items that were cluttering up the space.
My function looks like this. I had to switch the quotes on top access denied and top page not found because there were quotes in the t() function.
Thanks.
Comment #33
naught101 commented@thoughmas: You're probably better off just disabling and uninstalling admin_menu in D6, and then re-unstalling after the upgrade.
Comment #34
priya.chat commentedThanks #2 jason.fisher, your solution worked for me .
Comment #35
bohus ulrychThanks #2 @jason.fisher
I wanted to hide these menu items to my users.
I made just one small modification to avoid warning messages in watchdog