Please review with feedback [Archive]

jerrill

11-11-2005, 12:52 PM

Please review the basic structure of my website http://members.chello.nl/j.bangafoe/

I'm not a webdeveloper and coming this far with dreamweaver took a great while.
The site is made with tables which is not the right thing todo as i read from forum guru's. But i wonder if i have to start again in css or if there's a method for converting the current site to css.

I also wanted to have thumbnails next to the centre image. Clicking on a thumbnail would display a bigger more detailed image in the centre, but this is were i'm stuck at. I could us some advise.

ray326

11-11-2005, 02:47 PM

Your design is nice and clean so making the HTML right is pretty simple. E.g. here's my version of your second page. As you can see, stripping the tables is a piece of cake.

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
"http://www.w3.org/TR/html4/strict.dtd">
<html lang="en">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<meta name="Content-Script-Type" content="text/javascript">
<meta name="Content-Style-Type" content="text/css">
<title>Conforming HTML 4.01 Template</title>
<link rel="stylesheet" type="text/css" href="" />
<style type="text/css">
html, body {
margin:0;
padding:0;
border: 0;
background:#fff;
color:#000;
font-family: Arial, Helvetica, sans-serif;
font-size:1.0em;
}
#content {
height: 690px;
margin: 10px;
text-align: center;
background: #000;
color: #fff;
}
#content h1 {
font-size: 120%;
font-weight: bold;
padding-top: 1em;
}
#content ul {
list-style: none;
padding-top: 1em;
}
#content ul li {
display: inline;
font-size: 80%;
font-weight: bold;
}
#content ul li a {
color: #fff;
text-decoration: none;
}
</style>
<script type="text/javascript">
</script>
</head>
<body>
<div id="content">
<h1>Jerrill Bang-Afoe freelance Fotograaf</h1>
<img src="http://members.chello.nl/j.bangafoe/j.bangafoe/images/snesi-page2.jpg"
alt="Jerrill Bang-Afoe freelance Fotograaf"
title="Jerrill Bang-Afoe freelance Fotograaf">
<ul>
<li><a href="page%205%20disclaimer.html">Disclaimer</a> | </li>
<li><a href="page%206%20voorwaarden.html">Voorwaarden</a> | </li>
<li><a href="page%207%20werkwijze.html">Werkwijze</a> | </li>
<li><a href="mailto:snesi@chello.nl">Contact</a> | </li>
<li>© J.C.Bang-Afoe</li>
</ul>
</div>
</body>
</html>

I played with another version where the center image is actually in the background. In that case putting a list of thumbnail links on top of it would be pretty simple, too.