Design und Code bewerten (PORTFOLIO)

[Cuxi]

Aufbau der Kategorien beurteilen (FORUM)
Hallo Community,

da ich ja, wir Ihr vielleicht wisst, meinem Portfolio ein redesign verpasst habe würde ich auch dafür gerne eine bewertung bekommen.

Link: www.larsschuett.de

Also würde ich mich freuen wenn ihr den Code und auch das Design bewerten könntet.

Danke und Gruß
Cuxi

 

[fiedel]

Wo ist denn der Link?

 

[Cuxi]

upps...

editiert

 

[fiedel]

Achso! Na das kenn ich doch! Wusst nur nicht mehr, dass es von dir ist!

Code:
Wirklich sehr schöner Quellcode, du hast meiner Meinung nach alles richtig semantisch ausgezeichnet.

Design:
Wie bereits im anderen Thread erwähnt: Top!

Rechtschreibfehler (zähle ich mal zum Design dazu):
Endlich was, das nicht perfekt ist!

> Home:

Schön, dass Du es auf mein Portfolio geschafft hast.
Hier kannst Du Dir meine Werke beschauen und wenn Du möchtest
auch etwas bestellen. Nimm Dir Zeit und schaue einfach
mal durch. Und wenn Du auch fleißig twitterst, dann schaue
doch mal unten vorbei.

Da Du natürlich nicht nur zum Bildergucken hier sein möchtest,
stelle ich Dir hier exklusiv zwei psd's zur freien Verfügung.
Diese werden monatlich aktualisiert, sodass Du am Ende des Jahres
satte 24 psd.'s von Deinem Lieblingsdesigner hast.

Die Stammgäste werden es vielleicht bemerkt haben,
Larsschuett.de hat ein neues Gesicht! Hier nochmal die V2 Version
Nun würde ich gerne erfahren, ob sich die Sache Deiner Meinung
nach gelohnt hat, oder ein Flopp war.

> Portfolio:

hier siehst du alle meine Werke,
die schön genug waren um Online gestellt zu werden.
Schaue doch mal durch und wenn Dir was gefällt, schreibe
mich doch einfach mal an.

...so, das genügt erstmal, aber in den anderen Unterseiten sind auch noch ein paar Fehler (hauptsächlich Kommafehler) versteckt!

 

[Cuxi]

Vielen Dank dafür. Die Fehler habe ich sofort behoben. Evtl. gibt's ja noch mehr zu bemängeln.

Beste Grüße,

Cuxi | L. Schütt

 

[neuroleptika]

#kind-of-design a {
...
height:23px;
...
}

#contact-buttons a {
...
height:28px;
padding-top:5px;
...
}

Die Boxen laufen so bei einfacher Schriftvergrösserung über.

 

[SimonRue]

Hab grad bisl durchgeblättert.
Mir ist eine Klitzekleinigkeit beim Haftungsauschluss aufgefallen.
Der Hintergrund des Bildes (http://www.larsschuett.de/images/shadow-top.png) passt nicht ganz genau zum Hintergrund der Seite.
Aufgefallen im Mozilla 3.0.4

 

[Timmer]

Wirklich sehr tolles Design und toller Code!

Habe dich mal in DeviantArt geaddet.

Interesse, mir ein schlichtes Blogdesign zu machen?

 

[francesco]

Das Design gefällt mir sehr gut. Das Bookmark-Icon ist auch gelungen! :-) Allerdings finde ich die Schriftgröße beim Fließtext arg klein.

Zum Code:

Anstatt das br-Element für Layoutzwecke zu missbrauchen

<p> ... <br />
    ... <br />
    ... <br />
    ... <br />
</p>

würde ich einfach dem jeweiligen p-Element die nötige Breite zuweisen.

Das Einzige, was es noch zu bemängeln gibt, sind Geschichten wie

<div class="cleaner"></div>

oder

<div class="border"><!-- Dieser Kommentar ist nötig, damit der IE das div richtig darstellt.. --></div>

Die haben, wie du ja sicher weißt, null Semantik. Was passiert denn, wenn du der Klasse container auf der Startseite das clear: both; verpasst und das div-Element mit der cleaner-Klasse löscht?

P. S.: Wieso bindest du neun CSS-Dateien ein?

 

[Asterixus]

Die Schriftgröße ist in px ausgedrück. Das heißt, dass sie in manchen Browsern (IE6 und tiefer u.a.) nicht vergrößert werden kann.

Wo ist <h1>? Ich sehe nur <h2>. Die Überschriftenordnung ist nicht wirklich so, wie sie sein sollte. Überschrift erster Ordnung an dann erst zweiter!

<?xml version="1.0" encoding="UTF-8"?>
Warum benutzt du das? Das versetzt den IE in den Quirksmodus.

Du solltest Höhenangaben benutzten bei Bildern. Sei es im HTML (was selbst in deiner XHTML-Version erlaubt ist) oder im CSS.

Alternativtexte bei Bildern sind nicht da, damit die Seite valide ist. Die muss man nicht leer lassen, sondern richtig ausfüllen.


Aber nichts desto trotz, eine sehr gelungene Seite. Die Farben sind für meinen Geschmack etwas stark, aber im Ganzen passt alles sehr gut zusammen :)

 

[Basti93]

<?xml version="1.0" encoding="UTF-8"?>
Warum benutzt du das? Das versetzt den IE in den Quirksmodus.

Warum genau er das benutzt, weiß ich natürlich nicht sicher.
Allerdings gibt es einige Fälle , bei denen der Quirksmodus sinnvoll sein kann, um Fehler zu beheben oder Funktionen nachzubauen.
Beispiel: Min- und Max-width auch für den Internet Explorer (Schau ganz unten bei den "Weitere Voraussetzungen")