hi,
at the moment there is an extra a tag that is positioned over the bullet icon of the list. that approach can have theming issues.
in my opinion it would be better just to use the extra a tag for the icon (background-image), disable the bullet icon.
In the 7.x version there is also inline code that moves the a tag, so you could not even override that because inline priority > *:
element.style {
left: -16px;
margin-left: 21px;
}
as i remember the 6.3x version did not do that.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | expand_on_bullet-1176834-2.patch | 1.39 KB | ckng |
Comments
Comment #1
marcoka commentedso somehow the above code is the one that creates the inline code (with hardcoded sizes, i uncommented the lines)
(info: there is already a patch applied, the if statment checking hasClass('dhtml-menu'); , without that it kills a lot of stuff in views and others)
Comment #2
ckngAttached patch to correct the margin calculation and a better way to theme the .dhtml-folder
Somehow fixes #1380498: Admin options are disabled when "expand on bullet" is selected for me also, didn't investigate further.
Comment #3
ckngComment #4
ckngSomehow the patch in #2 is missing from d.o
Comment #5
twooten commentedThe patch in #4 fixed my issues.
Thanks!
Comment #6
Rory commentedThe patch in #4 also fixed my issues. I had to apply the !important rule to get a CSS style to apply properly from the dhtml_menu.css file though, as menu items began showing bullets afterwards.
So specifically from the patch:
became:
Comment #7
marcoka commentedafter nothing happening,i wrote http://drupal.org/project/dynatree
using an external script having the same name as the module, pretty solid.
Comment #8
mvcpatch #4 fixed this and #1380498: Admin options are disabled when "expand on bullet" is selected for me using 7.x-1.0-beta1. thanks!
Comment #10
vuilCommitted to the latest 7.x-1.x-dev branch.
Comment #11
vuilGive some credits about the issue. Thanks to everyone!
Comment #13
vuil