When "Collapse fieldsets on modules page" is enabled, the fieldsets collapse properly on admin/build/modules but do not collapse on admin/build/modules/list (which as near as I can tell is the exact same page, and linked to from Admin Menu)
Assuming that this isn't by design, here is a patch that corrects the behavior.
(It's possible that this is by design, and that admin/build/modules/list is supposed to show all modules uncollapsed. If this is the case, the wording on the link to the that page should be changed from "List" to "Show all" or something.")
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | admin_menu-479922-2.patch | 601 bytes | danepowell |
| admin_menu_collapse.patch | 629 bytes | danepowell |
Comments
Comment #1
sunThanks!
We want to use strpos() here instead.
Comment #2
danepowell commentedFair enough :)
Comment #3
sunOops. You should have mentioned that you already considered to use strpos(), but that would also apply to admin/build/modules/uninstall then. ;)
Original patch is ready to go then. :P
Comment #4
sunThanks for reporting, reviewing, and testing! Committed to all 3.x branches.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.
Comment #5
dave reid@danep: FYI, per the coding standards, make sure your booleans like 'true' are in all caps like 'TRUE'. Just so you know. :)
Comment #6
danepowell commented@Dave: I'm definitely still learning, thanks for the tip :)
Comment #8
gpk commentedHmm I used to use the List link to view all the modules in uncollapsed fashion. Useful when you can't remember which section a module lives in.
Not the end of the world I suppose, will just have to keep flipping the checkbox on the settings page when I need to view the complete list.
Comment #9
danepowell commentedMaybe a simple "expand all" link is what is needed there. You should probably open another issue if you're looking for something like that.