Closed (fixed)
Project:
Administration menu
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
21 Apr 2008 at 23:24 UTC
Updated:
3 Aug 2008 at 19:22 UTC
Jump to comment: Most recent file
Comments
Comment #1
sunHm. Yes, I've seen this issue myself before. However, I'm still tinkering about whether this could be fixed once for all contrib modules in admin_menu (instead of in multiple other contrib modules).
Comment #2
forngren commentedIs a more generic solution possible? Simple menu breaks the layout in a similar way in D6.
EDIT: okey, found the theme_ functions, so problem solved
EDIT 2: well, problem solved for me, but this should really be fixed by default.
Comment #3
sun@forngren: Do you want to let us know what you have changed to solve this?
Comment #4
forngren commentedIt's not pretty, but gets the job done. It would be smarter to include the correct (i.e. drupal core etc) scripts rather than blacklist the bad ones (simple menu). I will probably figure some more after I get some sleep.
Ohh, and btw, this line
should probably be changed to:
Comment #5
sunMoving over to admin_menu's queue.
Comment #6
sunThis is just the JS patch to prevent DAM from applying a margin-top if it's not output at all.
Comment #7
pwolanin commentedpatch that adds the new function plus above JS change.
Comment #8
pwolanin commentedok, we can simplify this for other modules, so they don't need to pass an argument.
See corresponding issue for advanced_help: http://drupal.org/node/285020
Comment #9
sunCommitted to HEAD, 6.x, and 5.x, thanks!
Comment #10
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.