Hi,
You have a lot of errors on that page that you should fix
Pop over the w3c html validator and get it validated before you move on with this. Don't worry we all have to start somewhere but the most important thing is to use the correct code logially and semantically.
Regarding the rollover then here is the updated code:
Code:
.outer {
width:480px;
margin:auto;
position:relative;
overflow:auto;
}
.image-holder {
float:right;
margin:0 8px 10px 0;
width:350px;
height:250px;
background:#fffccc;
border:1px solid #000;
padding:3px;
position:relative;
z-index:1;
display:inline;
padding:10px 10px 10px;
}
ul.links {
list-style:none;
margin:0 20px;
padding:0;
width:40px;
}
.links li {
width:40px;
list-style-type: none;
}
.links li a {
display:block;
width:40px;
padding:5px;
text-decoration:none;
}
.links li a span, .links li a b {
position:absolute;
right:19px;
top:-999em;
width:350px;
height:250px;
z-index:2;
overflow:hidden
}
.links li a span img {
display:block;
margin:0 0 5px
}
.links li a:hover, .links li.over a { visibility:visible; }
.links li a:hover span, .links li.over a span { top:11px; }
Make sure your html is well formed as you have many missing end tags and broken and invalid structures as mentioned above.
As an example the bottom of the page should look like this:
Code:
<script type="text/javascript">
<!--//--><![CDATA[//><!--
startList = function() {
var sfEls = document.getElementById("over").getElementsByTagName("LI");
for (var i=0; i<sfEls.length; i++) {
sfEls[i].onmouseover=function() {
//first remove all existing classes of .over
for (var j=0; j<sfEls.length; j++){
sfEls[j].className=sfEls[j].className.replace(new RegExp(" over\\b"), "");
}
this.className+=" over";// now add class
}
}
}
// addLoadEvent
function addLoadEvent(func) {
var oldonload = window.onload;
if (typeof window.onload != 'function') {
window.onload = func;
} else {
window.onload = function() {
if (oldonload) {
oldonload();
}
func();
}
}
}
addLoadEvent(startList);
//--><!]]>
</script>
<script type='text/javascript' src='http://insurancecommissionrefund.com/testwebsite/wp-includes/js/jquery/ui/jquery.ui.core.min.js?ver=1.8.16'></script>
<script type='text/javascript' src='http://insurancecommissionrefund.com/testwebsite/wp-includes/js/jquery/ui/jquery.ui.widget.min.js?ver=1.8.16'></script>
<script type='text/javascript' src='http://insurancecommissionrefund.com/testwebsite/wp-includes/js/jquery/ui/jquery.ui.tabs.min.js?ver=1.8.16'></script>
</body>
</html>
You have two or three html and body closing tags plus odd p elements all around the place. There is no need to wrap everything in a table either.
Also don't invent your own elements.
Code:
<SideBar>General Purpose Boats</SideBar>
There is no html element called SideBar!
You would need to use a class instead.
Code:
<h3 class="SideBar">General Purpose Boats</h3>
As the text is a heading then it should be a heading tag at the correct level (it depends on document structure so the h3 may not be the correct level depending n the rest of the page so keep it logical).
I'm not sure why you have so much code in that page for such a simple site.
Bookmarks