Need to add white bottom border to each block in this css menu.

[basslion]

The following is the css code i am using along with the html code. I would like to add a 2px bottom border to each menu item so it seperates each blue box with a white line. can someone quickly paste this into their editor and see what im doing wrong?

Here is the CSS code:

.suckerdiv ul{
margin: 0;
padding: 0;
list-style-type: none;
FONT-SIZE: 12px;
COLOR: #FFFFFF;
FONT-FAMILY: Tahoma, Verdana;
FONT-WEIGHT: bold;
background: #00256B;
width: 215px; /* Width of Menu Items */

}

.suckerdiv ul li{
position: relative;
padding-top: 4px;
padding-bottom: 5px;
}

/*Sub level menu items */
.suckerdiv ul li ul{
position: absolute;
width: 170px; /*sub menu width*/
background: #00256B;
top: 0;
visibility: hidden;
}

/* Sub level menu links style */
.suckerdiv ul li a{
display: block;
overflow: auto; /*force hasLayout in IE7 */
color: white;
text-decoration: none;
background: #00256B;
padding: 1px 5px;
border-bottom: 0;
}

.suckerdiv ul li a:visited{
color: white;
}

.suckerdiv ul li a:hover{
background-color: #00256B;
text-decoration: underline;
}

.suckerdiv .subfolderstyle{
background: url(media/arrow-list.gif) no-repeat center right;
}


/* Holly Hack for IE \*/
* html .suckerdiv ul li { float: left; height: 1%; }
* html .suckerdiv ul li a { height: 1%; }
/* End */
-->
</style>
<script type="text/javascript">

//SuckerTree Vertical Menu 1.1 (Nov 8th, 06)
//By Dynamic Drive: http://www.dynamicdrive.com/style/

var menuids=["suckertree1"] //Enter id(s) of SuckerTree UL menus, separated by commas

function buildsubmenus(){
for (var i=0; i<menuids.length; i++){
var ultags=document.getElementById(menuids[i]).getElementsByTagName("ul")
for (var t=0; t<ultags.length; t++){
ultags[t].parentNode.getElementsByTagName("a")[0].className="subfolderstyle"
if (ultags[t].parentNode.parentNode.id==menuids[i]) //if this is a first level submenu
ultags[t].style.left=ultags[t].parentNode.offsetWidth+"px" //dynamically position first level submenus to be width of main menu item
else //else if this is a sub level submenu (ul)
ultags[t].style.left=ultags[t-1].getElementsByTagName("a")[0].offsetWidth+"px" //position menu to the right of menu item that activated it
ultags[t].parentNode.onmouseover=function(){
this.getElementsByTagName("ul")[0].style.display="block"
}
ultags[t].parentNode.onmouseout=function(){
this.getElementsByTagName("ul")[0].style.display="none"
}
}
for (var t=ultags.length-1; t>-1; t--){ //loop through all sub menus again, and use "display:none" to hide menus (to prevent possible page scrollbars
ultags[t].style.visibility="visible"
ultags[t].style.display="none"
}
}
}

if (window.addEventListener)
window.addEventListener("load", buildsubmenus, false)
else if (window.attachEvent)
window.attachEvent("onload", buildsubmenus)

</script>

Here is the HTML Code:

<div class="suckerdiv">
<ul id="suckertree1">
<li><a href="#">Item 1</a></li>
<li><a href="#">Item 2</a></li>
<li><a href="#">Folder 1</a>
<ul>
<li><a href="#">Sub Item 1.1</a></li>
<li><a href="#">Sub Item 1.2</a></li>
</ul>
</li>
<li><a href="#">Item 3</a></li>

<li><a href="#">Folder 2</a>
<ul>
<li><a href="#">Sub Item 2.1</a></li>
<li><a href="#">Folder 2.1</a>
<ul>
<li><a href="#">Sub Item 2.1.1</a></li>
<li><a href="#">Sub Item 2.1.2</a></li>
<li><a href="#">Sub Item 2.1.3</a></li>
<li><a href="#">Sub Item 2.1.4</a></li>
</ul>
</li>
</ul>
</li>
<li><a href="#">Item 4</a></li>
</ul>
</div>

[Cars-blog]

Why not just add this code to your css?

.suckerdiv a { border-bottom: solid 2px #fff;}

[basslion]

i assume you didnt test it out? it didnt work for me

it seems it should work.

[Costin Trifan]

in here:

.suckerdiv ul li{
position: relative;
padding-top: 4px;
padding-bottom: 5px;
border-bottom: solid 2px #fff;
}

now it should work

[basslion]

works like a charm, thanks !!!

Now with the flyout subcats, how could i put a left side border of white so it seperates the subcats from the top cats with a white line?

[Costin Trifan]

in here

/*Sub level menu items */
.suckerdiv ul li ul {
position: absolute;
width: 170px; /*sub menu width*/
background: #00256B;
top: 0;
visibility: hidden;
border-left: solid 2px #fff;
}

[basslion]

Thanks again, worked perfect

Appreciate your help sir.