molily: Bitte Projekt bewerten!

Beitrag lesen

Was für ein Codegewühl:

<form action="mailto:email@profi-dozent.de" name="formular" method=post>
    <table width="400" align="center" border="0" cellpadding="14" cellspacing="0">
     <tr>
      <td class="tabelle">
       Name:<img src="img/spacer.gif" width="42" height="10" alt=""><input type="text" name="name" size="42" style="background-color: #FFFFFF; border-left: medium none #39445B; border-right: medium none #39445B; border-top: medium none #39445B; border-bottom: 1px solid #39445B">
       <br>
       Telefon:<img src="img/spacer.gif" width="35" height="10" alt=""><input type="text" name="telefon" size="42" style="background-color: #FFFFFF; border-left: medium none #39445B; border-right: medium none #39445B; border-top: medium none #39445B; border-bottom: 1px solid #39445B">
       <br>
       E-Mail:<img src="img/spacer.gif" width="40" height="10" alt=""><input type="text" name="e-mail" size="42" style="background-color: #FFFFFF; border-left: medium none #39445B; border-right: medium none #39445B; border-top: medium none #39445B; border-bottom: 1px solid #39445B">
       <br><br><br>
       <strong>Ihre Anfrage:</strong>
       <br><br>
       <textarea name="anfrage" rows="20" cols="42" style="background-color: #FFFFFF; border-left: 1px solid #39445B; border-right: 1px solid #39445B; border-top: 1px solid #39445B; border-bottom: 1px solid #39445B; scrollbar-3dlight-color: #5E7CAD; scrollbar-arrow-color: #5E7CAD; scrollbar-base-color: #5E7CAD; scrollbar-darkshadow-color: #5E7CAD; scrollbar-face-color: #FFF8ED; scrollbar-highlight-color: #FFFFFF"></textarea>
       <br>
       <input type="image" name="absenden" src="img/senden.gif" alt=" Anfrage absenden ">
       <br>
      </td>
     </tr>
    </table>
   </form>

Das räumen wir auf und daraus wird:

<form action="kontakt.cgi" name="formular" method="post">
<table border="0" cellpadding="4" cellspacing="0" summary="" id="kontaktformular">
<tr>
<th><label for="name">Name:</label></th>
<td><input type="text" name="name" id="name" size="42"></td>
</tr>
<tr>
<th><label for="telefon">Telefon:</label></th>
<td><input type="text" name="telefon" id="telefon" size="42"></td>
</tr>
<tr>
<th><label for="e-mail">E-Mail:</label></th>
<td><input type="text" name="e-mail" id="e-mail" size="42"></td>
</tr>
<tr>
<td colspan="2">
<p><label for="anfrage"><strong>Ihre Anfrage:</strong></label></p>
<textarea name="anfrage" rows="20" cols="42"></textarea>
</td>
</tr>
<tr>
<td colspan="2"><input type="submit" name="absenden" value="Anfrage absenden" id="submit"></td>
</tr>
</table>
</form>

Den Rest lösen wir mit CSS, etwa so:

#kontaktformular th, #kontaktformular td {padding:0 0 8px 0; font-size:14px; font-weight:normal; text-align:left;}
#kontaktformular th {width:70px;}
#kontaktformular {border-spacing:0; border-collapse:collapse; border-style:none;}
#kontaktformular p {margin:10px 0 20px 0;}
#kontaktformular input {color:black; background-color:white; border:1px solid #39445B; width:280px;}
input#submit {width:200px;}
#kontaktformular textarea {color:black; background-color:white; border:1px solid #39445b; width:350px; scrollbar-3dlight-color: #5e7cad; scrollbar-arrow-color: #5e7cad; scrollbar-base-color: #5e7cad; scrollbar-darkshadow-color: #5e7cad; scrollbar-face-color: #fff8ed; scrollbar-highlight-color:#fff;}

Hast du einmal versucht, die Navigationslisten mit ul-Elementen auszuzeichnen? Das würde auch viel Code sparen.

Die title-Elemente aller Dokumente der Site sind gleich, das solltest du ändern.

Welchen Sinn erfüllen die Validitäts-Buttons? Deinen »Website Design By«-Button würde ich auch nicht auf jedem Dokument wiederholen...

<div id="allgemein">
   <br><br><br><br><br><br><br><br>
    Andreas Plagge     <img src="img/punkt.gif" width="4" height="11" alt="">     Weyrichstraße 6a     <img src="img/punkt.gif" width="4" height="11" alt="">     64625 Bensheim     <img src="img/punkt.gif" width="4" height="11" alt="">     E-Mail: <a class="email" onfocus="this.blur()" href="mailto:email@profi-dozent.de">email@profi-dozent.de</a>
    <br><br>
  </div>

Wie wäre es mit etwas CSS anstatt dutzender <br>? Das gilt quasi für den ganzen Seitenaufbau.

Mit onfocus="this.blur()" tötest du jede Tastatur-Bedienbarkeit im Keim.

Anstatt href="index.htm" könntest du href="./" schreiben, dann ist die Startseite nur einmal vorhanden.