Los Borders no funcionan como esperaba... lección 3 curso paso a paso

Hola a todos, estoy iniciándome en esto del diseño web, hasta ahora solo estoy en la lección 3 del curso, tratando de ponerle bordes a las distintas capas, pero cuando le pongo el borde a la capa de contenido aparece como separado por los párrafos o algo así, si alguien me ayuda lo agradeceré mucho...

mi web...

http://www.webdealbitha.comocreartuweb.es/plantilla.html

mi codigo...

plantilla

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">

<html>
<head>
<title>Page title</title>

<link rel="stylesheet" href="estilo-general.css" type="text/css" media="all">
</head>
<body>
<div id="global">
<div id="curva-superior"></div>
<div id="cabecera">Bienvenidos a mi Web</div>
<div id="navegacion">Barra de Navegación</div>

<div id="contenido">
<div id="menu">
<ul><li><a href="#">Enlace 1</a></li><li><a href="#">Enlace 2</a></li><li><a href="#">Enlace 3</a></li>
</div>
<h1>Esta será la zona principal de la web</h1>
<p>Este es mi segundo parrafo solo para probar como se comporta mi pagina web
debo ser cuidadosa a la hora de colocar los puntos y comas y sobre todo
de no escribir sin colocar las etiquetas correspondients</p>
<p><h2>...Y bueno para seguir probando</h2>
escribiré este otro parrafo y veremos como se sigue
comportando el menu de la izquiera es bien importante saber que pasará</p>
</div>
<div id="pie"> este será el pie de página</div>
</div>

</body>
</html>


estilo-general


* {text-indent:0px; margin:0px; padding:0px; border:0px;}
body {text-align:center;}
#global {width:800px; margin:4px auto;}
#cabecera {background-color:white;border-left:black 1px solid; border-right:black 1px solid;}
#curva-superior {background-image:url(objetos/curva-superior.gif);
background-repeat:no-repeat; width:800px; height: 12px;
overflow:hidden;}
#navegacion {background-color:gray; border:black 1px solid;}
#contenido {background-color:orange;}
#contenido p {padding:5px 10px 5px 10px; text-indent:15px;
border-right:black 1px solid;border-left:black 1px solid; }
#menu {background-color:yellow; width:150px; float:left; text-align:left; margin:10px 10px 10px 10px;}
#menu li {list-style:none ; margin: 4px 0px 4px 6px;}
#pie {background-color:brown;}
a {color:brown; text-decoration:none; display:block;}
a:link {}
a:visited {}
a:hover {color:red; text-decoration:underline; background-color:silver;}
a:active {}
p {text-align:justify;}
h1 {font-size:1.2em; color:blue; font-weight:bold; text-decoration:underline; text-align:center;}
h2 {font-size:1.1em; color:blue; font-weight:bold; text-decoration:none; text-align:left;}
#curva-inferior {background-image:url(objetos/curva-inferior.gif);background-repeat: no-repeat ;
width:800px ; height: 12px overflow: hidden;}