• Jetzt anmelden. Es dauert nur 2 Minuten und ist kostenlos!

Design und Code bewerten (EDITOR)

Also Times New Roman geht mal gar nicht. Auch noch im Header... Nein, versuchs mal mit einer serifenlosen Schrift.
 
div-Suppe fällt mir dazu ein. Ich sehe da nichts ausser <div> und <a>, von Semantik keine Spur.

Dir fehlen sämtliche Überschriften und Textabsätze.

Such hier im Forum mal nach Semantik, da wirst du einiges zu finden.
 
Es ist keine Schriftart definiert. Somit sieht diese in jedem Browser anders und unterschiedlich groß aus.

Die Textblöcke sind nach meinem Geschmack auch zu breit. Dadurch erfordert das Lesen eine sehr hohe Konzentration. Sieh dir hierzu mal diese Tipps an: Schrift und Text optimieren
 
Muss meinen Vorrednern da Recht geben. Du hast da etwas falsch verstanden: DIVs sind zwar (grob gesagt) "gut", sollten aber nicht missbraucht werden!
Du packst ja um jedes Element was du hast ein eigenes div-Element, das geht gar nicht. Du hast da Überschriften, Textabsätze usw. und die gehören auch als solche ausgezeichnet.

Das Design mag mir leider auch nicht so sehr gefallen. Zwar mag ich allgemein schon schlcihte Layouts, aber bei dir fehlt mir doch noch ein KNiff oder so. Die Farben wirken sehr kühl.

MfG Icy
 
Ok danke!

Ich werd das erstmal anhand des Feedbacks ein wenig umbauen!

Hab jetzt mal viele der div's ersetzt, mit Elementen die mir als sinnvoll erscheint haben.
Logo hab ich auch ausgebessert. Besser?

Ansonsten, muss ich mir noch Verbesserungen zum Design überlegen.
 
Zuletzt bearbeitet von einem Moderator:
Ist ja schonmal ein kleiner Schritt, aber immer noch viele überflüssige Divs.
Beispiel:

HTML: 
    <div id="top">
      <div class="banner">
        <a href="http://www.daniel-zimmermann.at/Blog/">
          <img src="transparent.png" alt="logo" style="top: 0px; left: 25px; position: absolute; border:0" />
        </a>
      </div>
    </div>

Warum löst du das nicht einfach so:

HTML: 
<h1 class="top">
  <a href="http://www.daniel-zimmermann.at/Blog/"><img src="transparent.png" alt="logo" style="top: 0px; left: 25px; position: absolute; border:0" /></a>
</h1>

Und wenn du schon extra zwei verschiedene Klassen um ein Bild drumrumbastelst, warum packst du dann nochmal Style-Angaben in das img-Tag?

Edit:

HTML: 
<div id="menu">
  <div style="background-color:#B3DFFF; width: 100%">Menu</div>
  <br />
  <a href="login.php">Admin-Center</a>
</div>

Auch das ist nicht korrekt. Das solltest du als Liste auszeichnen mit (bisher) einem Listenelement.

MfG Icy
 
Zuletzt bearbeitet:
Icy schrieb:
Muss meinen Vorrednern da Recht geben. Du hast da etwas falsch verstanden: DIVs sind zwar (grob gesagt) "gut", sollten aber nicht missbraucht werden!
Zum Vergrößern anklicken....
Das selbe gilt auch für Tabellen, Textabsätze, Adressen, ...

Schmeiß außerdem noch die <br /> raus, die Du missbrauchst, um Abstände zu erzeugen. HTML ist nicht dafür da, die Darstellung zu ändern. Dafür ist CSS zuständig.

"Design" kann man da nicht bewerten, da ist keines. Ich mein...die Farben sind Komplementärfarben, die passen nicht zusammen...
 
Du musst dich einloggen oder registrieren, um hier zu antworten.

Neueste Beiträge

Zurück
Oben