Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.In style.css line 93 the following rule is definied:
ul, li.leaf {
list-style-image: url(images/menu-leaf.gif);
}
This rule adds an unexpected (and unwanted) bullet to all unordered lists (see attached screenshot or admin/content/node or admin/user/user).
Fix for this:
li.leaf {
list-style-image: url(images/menu-leaf.gif);
}
Stefan
| Comment | File | Size | Author |
|---|---|---|---|
| #15 | 356240_image_bullets_15.patch | 1.34 KB | stborchert |
| #6 | 356240_no_image_for_ul_6.patch | 782 bytes | stborchert |
| #3 | 356240_no_image_for_ul_3.patch | 732 bytes | stborchert |
| no_image_for_ul.patch | 750 bytes | stborchert | |
| list-style-image.jpg | 10.3 KB | stborchert |
Comments
Comment #1
stborchertOf course that patch needs review :-)
Comment #3
stborchertConverted line endings to LF (UNIX).
Comment #4
sivaji_ganesh_jojodae commentedit works :D
Comment #5
catchShouldn't this be
ul li.leafOtherwise it could apply to ol too right?
Comment #6
stborchertYes, you're right.
Changed the rule to
ul li.leaf.Comment #7
stborchertpatch needs review of course.
Comment #9
stborchertHuh? "Translation import" fails because of a CSS rule? Maybe a problem with the test bot (another patch of mine faild with the same result)?
Comment #10
stborchertSame patch (retry). I don't think a css rule let a translation import fail. Tests passes on to different servers of mine.
Comment #12
stborchertretest
Comment #13
johnalbinGarland, by design, wants to style all its UL lists in this way. So
ul, li.leafis the proper ruleset to accomplish that.I think the patch in #4 of #394670: Bullet on UL lists showing up incorrectly is the correct approach.
Comment #14
johnalbinComment #15
stborchertThis is the patch created by momendo with a small addition (user-admin-filter was missing).
Comment #16
johnalbinThe patch looks good.
But it looks like the real problem was that those forms were incorrectly adding leaf class to those list items. I no longer see the problem in Drupal 6.11 which means it was fixed elsewhere. Unfortunately, I can't find the issue where it was fixed or I would link to it.
Comment #17
stborchertIssue is still valid (setting both issues to duplicate didn't fix it :-) ).
Comment #19
stborchertDear bot. I'm sorry but I didn't trust you :-) Please re-test it.
Comment #20
eigentor commentedThis works.
Comment #21
webchickOMG! This has been annoying me forever. Thanks so much!
Committed to HEAD.
Comment #22
eigentor commentedA giant beer to stBorchert. Even with having set this to RTBC (was there bribing involved?) I only now fully realize that it annoyed me forever, too.