Li in navigation menu double dots;-)

sciman - August 21, 2008 - 17:35
Project:Oxidation
Version:6.x-1.0
Component:Code
Category:support request
Priority:normal
Assigned:DonCoryon
Status:needs work
Description

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. .

AttachmentSize
oxidationLi.png21.9 KB

#1

DonCoryon - August 22, 2008 - 19:22

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?

#2

sciman - August 23, 2008 - 01:00

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.

#3

DonCoryon - August 24, 2008 - 17:32

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.

#4

sciman - August 23, 2008 - 15:56

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

#5

sun - August 24, 2008 - 22:30
Status:active» won't fix

Seems like this issue cannot be fixed in this project.

#6

DonCoryon - August 25, 2008 - 15:33
Status:won't fix» closed

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.

#7

sciman - August 25, 2008 - 17:42

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

AttachmentSize
double-bullets.jpg 48.28 KB

#8

DonCoryon - August 25, 2008 - 23:20

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.

AttachmentSize
oxidation_navigation_firefox_internet_explorer.png 125.15 KB
oxidation_navigation_firefox_internet_explorer_after_patch.png 45.6 KB
oxidation-6.x-1.0_list_styling.patch 430 bytes

#9

DonCoryon - August 25, 2008 - 23:21
Status:closed» needs review

Changing status.

#10

DonCoryon - August 26, 2008 - 00:26
Assigned to:Anonymous» DonCoryon
Status:needs review» needs work

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.

AttachmentSize
oxidation_6.x-1.0_list_style_patch_2.patch 366 bytes
 
 

Drupal is a registered trademark of Dries Buytaert.