Currently, bullets are removed from the admin filter ul list with list-style-type: none. However, if you have used list-style-image to style ul bullets in your site, then the bullets will still show up because list-style-type does not override list-style-image. The easiest way to fix this is to change list-style-type to list-style in #node-admin-filter in node.css. The list-style property sets all the properties for a list in one declaration. In contrast, the list-style-type sets the type of list-item marker (ex. disc, circle, square, etc.).

I have made this css change on a few sites and thought that it might be a good modification to core. I've attached my very small patch.

Here's the old code:

#node-admin-filter ul {
  list-style-type: none;
  padding: 0;
  margin: 0;
  width: 100%;
}

And, here's the revised code:

#node-admin-filter ul {
  list-style: none;
  padding: 0;
  margin: 0;
  width: 100%;
}

Comments

pancho’s picture

Component: node system » theme system
Category: feature » bug
StatusFileSize
new2.17 KB

Good catch! The change makes sense!
I rerolled the patch changing this also in other occurences.
Please test...

JuliaKM’s picture

Thanks! I applied the patch and it worked perfectly.

jody lynn’s picture

Please take a look at http://drupal.org/node/197833 . When I looked into this issue it seemed to me that it's really because theme_node_filter_form is no longer being called. While your patch treats the symptom I'm concerned that the real cause is more complicated.

jody lynn’s picture

Status: Needs review » Closed (duplicate)

This is being taken care of at http://drupal.org/node/197833

JuliaKM’s picture

I'm not sure if http://drupal.org/node/197833 solves the issue. All that I am trying to do here is get the style to remove bullets changed from list-style-type: none; to list-style:none so that if you add images within a site using list-style-image, they are removed with list-style:none. If list-style-type:none stays, then list-style-type:none will not override images.

As far as I can tell, http://drupal.org/node/197833 adds back in the id #node-admin-filter but it does not change the css from list-style-type:none to list-style:none.

pancho’s picture

Status: Closed (duplicate) » Needs review

According to JuliaKM, this is not fixed by #197833, so it's no duplicate. This needs to be checked.

dvessel’s picture

Status: Needs review » Closed (won't fix)
StatusFileSize
new7.69 KB

You're forgetting IE. Both 6 & 7 do not read list items the same way. There's a workaround but I don't think it's worth it. When using images for your list items, override it from your theme.

dvessel’s picture

StatusFileSize
new11.04 KB

wrong screen shot.

JuliaKM’s picture

Ok. Will do. Thanks for your help.