I'm a novice at css, but would like help in removing the double dots in the navigation menu. I'm not sure that my words here are optimum, so I'll attach a graphic. For bullet items in the menu there sometimes appears, in addition to the small graphic, an html sort of bullet which I've associated with li tags. I suspect there's a small bit of cleverness to be performed with the css? Any tips would be much appreciated. Thanks for such a nice theme. .

CommentFileSizeAuthor
#10 oxidation_6.x-1.0_list_style_patch_2.patch366 bytesCory Goodwin
#8 oxidation_navigation_firefox_internet_explorer.png125.15 KBCory Goodwin
#8 oxidation_navigation_firefox_internet_explorer_after_patch.png45.6 KBCory Goodwin
#8 oxidation-6.x-1.0_list_styling.patch430 bytesCory Goodwin
#7 double-bullets.jpg48.28 KBsciman
oxidationLi.png21.9 KBsciman

Comments

Cory Goodwin’s picture

Cory Goodwin commented

It does not do that on my screen or in any of my browsers. There must be a root cause and if we track it down we can fix it. First question, are you using some type of menu altering module?

If so, if you disable the module (or modules) does the problem go away?

sciman’s picture

sciman commented

Yes, I did have a number of modules running. I turned of dhtl menu, taxonomy menu, active menus, and a menu module.

I went to performance, turned off all caching, cleared all cache. The problem remains in both Firefox and Safari.

I notice that this happens in the Administer Block, which has 9 items at the next level, and some of course rather steep. By default the items are collapsed, though when toggled open, they do not close. Click a longer one, and lessers will collapse. Some items have several bullets, and some 3.

Cory Goodwin’s picture

Cory Goodwin commented

A module or some other change to Drupal is adding the "plus/minus" boxes so you need to continue looking for the root cause because Oxdiation does not style the menus. The only styling is from the system-menus.css.

The next step would be to either deactivate all modules or install Oxidation on a clean install. Then activate each module one at a time until the problem is reproduced. This will allow you to narrow down which module it is.

sciman’s picture

sciman commented

Thanks Don. I'll have a go at it, and if I find anything, I'll let you know.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Active » Closed (won't fix)

Seems like this issue cannot be fixed in this project.

Cory Goodwin’s picture

Cory Goodwin commented
Status: Closed (won't fix) » Closed (fixed)

I have not been able to reproduce the problem. I could be wrong, but I believe the error is caused by another module. In order to help solve the problem the root cause needs to be discovered.

sciman’s picture

sciman commented
StatusFileSize
new double-bullets.jpg48.28 KB

It looks like Oxidation is not overriding the Drupal style defined in system-menus.css

Drupal sets list-style-image and list-style-type, but Oxidation is setting a background image and not un-setting the styles declared in system-menus.css.


style.css?4 (line 354)

#sidebar li li {
  background:transparent url(images/img06.gif) no-repeat scroll left center;
  padding-left:15px;
}


modules/system/system-menus.css?4 (line 17)

li.collapsed {
  list-style-image:url(../../misc/menu-collapsed.png);
  list-style-type:disc;
  margin:0;
  padding:0.2em 0.5em 0 0;
}

Steve

Cory Goodwin’s picture

Cory Goodwin commented
StatusFileSize
new oxidation-6.x-1.0_list_styling.patch430 bytes
new oxidation_navigation_firefox_internet_explorer_after_patch.png45.6 KB
new oxidation_navigation_firefox_internet_explorer.png125.15 KB

The reason why I was unable to reproduce the bug, is because I didn't notice you had the menu in the right sidebar. It also didn't dawn me that the left and right sidebars had different list styling. More accurately, that the list styling was reserved for only the right sidebar, which is why when I inspected the list elements, nested in the left sidebar, no Oxidation styling showed.

You were correct that Oxidation was the culprit. I went ahead and removed all of the list styling since it doesn't really change the overall look of the theme.

Patch attached.

Cory Goodwin’s picture

Cory Goodwin commented
Status: Closed (fixed) » Needs review

Changing status.

Cory Goodwin’s picture

Cory Goodwin commented
Assigned: Unassigned » Cory Goodwin
Status: Needs review » Needs work
StatusFileSize
new oxidation_6.x-1.0_list_style_patch_2.patch366 bytes

Or we can keep the Oxidation bullets with this patch.

However, I have company coming over tonight, so I need to get going, but this works in Opera, FF, but not Safari and IE. Code needs more work.

Cory Goodwin’s picture

Cory Goodwin commented
Status: Needs work » Patch (to be ported)
Cory Goodwin’s picture

Cory Goodwin commented
Status: Patch (to be ported) » Closed (fixed)

changes in dev version