Hey Jungs und Mädels!

Hab gerade einen Kundenauftrag fertig gemacht und wollte eure Meinung zum Code wissen! Entdeckt ihr irgendwelche Fehler, die ich in der Hektik vergessen oder verdrängt habe? Was könnte man noch verbessern?
Wie gesagt, es geht mir nur um den Code, da das Design vom Kunden als PSD vorgegeben wurde.

Gemacht wurde:
+ komplette Anpassung in HTML und CSS
+ Erstellung einer geänderten Newsansicht
+ Erstellung der LastForum Box
+ Erstellung der LastWars Box (noch nicht zu sehen! unter testseite.pixelphysics.de wäre die LastWars Box zu sehen)
+ Erstellung des Sliders
+ Einbindung eines erweiterten PicofX Moduls inkl. Lightbox (unter dem Content)
+ Lightbox auf alle Bilder die via BBCode eingebunden werden (also auch Forum, news usw.)
+ komplette Umcodierung der Login- und Searchbox

MfG Robert Seitz

betroffene Homepage: angelus-mortis.com

Schöne Arbeit.
Insgesamt kriegt es eine Note 1- von mir. (Oder A-)

Warum?

Gut ist: Schön und Schlicht Gehaltenes Design, Farben der Schrift und die Schriftart selber gute Auswahl.

Schlecht: Für einen Clan finde ich die "Enterpage" am Anfang etwas "zu" unheimlich, bzw finde ich das dies gar nichts mit einem Clan zutuen hätte, aber wenn man´s mag dann ist es ja okay.

Info: Ich weis nicht wie es bei euch ist aber bei einer Auflösung von 1024 x 768 ist die Seite
etwas zu groß geraten. Ich jetzt hier auf meinem "Neuen Mac" ist die Seite von 1366 x 768 perfekt. Also wirklich großes und schönes Design.

Wenn die Enterpage am Anfang nicht von dir ist dann sage es.
Ich werde das Schlechte dann raus edieren und eine 1+ draus machen

Mit freundlichen Grüßen,
Lukas

Zitat geschrieben von Kaite


Wenn man das OS von Apple auf einem Laptop installiert der auf Windows basiert, ist das kein Mac... merk dir das!


@Robert0506
Also mir gefällt die Umsetzung vom Slider und von der Navi.
Welches Script hast du für den Slider und die Navi verwendet?

Hey, sieht gut aus muss ich sagen. Nur mich oder eher gesagt mein vadilator stört etwas ^^:

Result: 2 Fehler / 103 Warnungen
line 6 column 1 - Warnung: <meta> element not empty or not closed
line 169 column 7 - Warnung: <input> element not empty or not closed
line 169 column 301 - Warnung: <br> element not empty or not closed
line 168 column 7 - Warnung: missing </form>
line 171 column 7 - Warnung: discarding unexpected </div>
line 172 column 9 - Warnung: <input> isn't allowed in <tr> elements
line 166 column 5 - Info: <tr> previously mentioned
line 173 column 9 - Warnung: <a> isn't allowed in <tr> elements
line 166 column 5 - Info: <tr> previously mentioned
line 173 column 58 - Warnung: <img> element not empty or not closed
line 174 column 7 - Fehler: discarding unexpected </form>
line 175 column 7 - Fehler: discarding unexpected </td>
line 190 column 83 - Warnung: <div> isn't allowed in <table> elements
line 190 column 31 - Info: <table> previously mentioned
line 195 column 116 - Warnung: <input> element not empty or not closed
line 203 column 1 - Warnung: <img> element not empty or not closed
line 206 column 1 - Warnung: <img> element not empty or not closed
line 209 column 1 - Warnung: <img> element not empty or not closed
line 212 column 1 - Warnung: <img> element not empty or not closed
line 333 column 9 - Warnung: <img> element not empty or not closed
line 242 column 6 - Warnung: <div> anchor "newshead_title" already defined
line 243 column 9 - Warnung: <div> anchor "news_infos" already defined
line 245 column 5 - Warnung: <div> anchor "news_content" already defined
line 277 column 9 - Warnung: <div> anchor "news_infos" already defined
line 279 column 5 - Warnung: <div> anchor "news_content" already defined
line 280 column 6 - Warnung: <div> anchor "news_content_inhalt" already defined

(habe einige Rausgelöscht, damit hier nicht soviel gescrollt werden muss. Schau es dir am besten selbst an.)


Sind natürlich einige ilch standard dinger bei usw.
Aber nächstes mal vllt:
<img /> statt <img>
<br /> statt <br>
<input /> statt <input>
usw. Mögen die Browser glaube ich lieber.

Und id's in css sind eig für nur einfache Elemete wie...
<div id="header">...</div>
bei sachen wie
<div id="boxhead">
class benutzen. Hat soweit ich weiß, was mit dem Ladeverhalten der Browser zu tun, aber nur von der Geschwindigkeit her.

Sonst wie gesagt gut gemacht.

PS: bei der neusten Version von Opera stimmt was in der Last Forum Box nicht.

Zitat geschrieben von Tr3icio



ja, meinte ich eigentlich so. Aber bei stinkt normalem CSS und HTML funktionierts, aber sollte man nicht machen.

Danke nochmal fürs Feedback! Das mit den Klassen war ein guter Tipp! Vielen Dank!

Was ich persönlich nicht so ganz nachvollziehen kann, ist wieso ist mein Quellcode 100% Valide wenn ich ihn per FileUpload bei W3C hochlade, der Quellcode hat aber dann 51 Fehler, wenn ich die Homepage online teste? Hat da jemand nen Tipp für mich?

Also ich hab bei Opera keine Fehler entdeckt!


Zuletzt modifiziert von Robert0506 am 18.07.2011 - 15:49:53

Jo war eigentlich von Grund auf CSS 3 valide. Habs aber grad nochmal getestet... Hat sich doch noch ein Fehler eingeschlichen! Aber hab den Fehler grad ausgebessert!