| von Efchen |
Gern geschehen. Jederzeit. Am besten in diesem Thread, damit ich die Benachrichtigungsmail bekomme  |
| von crazyhero |
@Efchen
Wow, danke für die mehr als ausführliche Kritik. Mein Code ist etwas unsauber, da er teilweise aus alten Codestücken bzw. Fremdcodestücken (Newsletter) besteht. Das muss ich noch alles anpassen.
Ich werde mir deine Kritik Punkt für Punkt durcharbeiten und wenn ich Fragen hab frag ich dich.
Danke nochmal! |
| von Efchen |
Meine Kritikpunkte. Bei Rückfragen einfach hier posten:
1. Es ist zwar lobenswert, dass sich die Seite der Fenstergröße anpasst, aber bei meinen 772x970px ist die Seite doch schon etwas arg schmal, zumal 1/3 des Platzes (der ja ohnehin schon knapp bemessen ist bei mir) einfach verschwendet wird.
2. Optisch finde ich die Site schlicht und schön. Der haftet zwar diese "CSS-Layout-Optik" an, aber ich persönlich kriege ohnehin nicht mehr als sowas auf die Beine, daher bin ich von sowas eigentlich auch angetan. Außerdem finde ich, weniger ist oft viel mehr.
3. Das Menü oben links ist etwas verloren.Der Fabrkontrast ist sehr niedrig, gegen den Rest der Site geht das unter. Und irgendwie schreit es nach einer eigenen Überschrift.
4. Bei den Google-Anzeigen gibt es bei den Links weder einen Hover-Effekt, noch sieht man das Linkziel in der Statuszeile. Auch der Mauszeiger ändert sich nicht. Daher weiß man nicht, ob es sich hierbei um Links oder um unterstrichenen Text handelt. Das verunsichert. Zumal ich nicht weiß, wo ich da lande, wenn das nicht in der Statuszeile steht.
5. Der letzte Gutscheinblock unter Douglas passt nicht in den Content-Bereich sondern gehts rechts darüber hinaus, und sogar aus dem Browserfenster raus.
6. Die Anzeige des Pageranks passt optisch nicht zum restlichen Design, und außerdem erachte ich das ohnehin nur als Angeberei. Ähnlich doof wie Counter.
7. Im Impressum fehlt die Telefonnummer und die Mailadresse (§6 TDG).
8. Beim Kontaktformular passt die textarea nicht in den Content-Bereich (erinnere Dich: Browser 772px breit). Die Seite ist so kurz, dass das Login über den unteren Rand hinaus geht. Und das Formular lässt sich leer abschicken. Warum die nächste Seite nicht mehr orange, sondern grün ist, verstehe ich nicht. Es verunsichert. Außerdem fehlt auf der Seite der <title>
9. Das im Impressum angegebene Copyright existiert nicht in Deutschland. Wir haben sowas hier nicht.
10. Nach neustem DIN/ISO weissnichtwieviel schreibt man das Länderkennzeichen in der Anschrift nicht mehr vor die PLZ, sondern schreibt das Land ausgeschrieben unter die PLZ/Stadt.
11. Der Code sieht schön, übersichtlich und modern aus. Sehr angenehm nach meinen zwei letzten Bewertungen Jedoch als Perfektionist solltest Du Links auf Verzeichnisse richtig schreiben. Sie werden mit abschließendem Slash geschrieben (also "/ebay/" statt "/ebay"). Das ist zusätzliche Arbeit für den Server, den Slash da anzuhängen.
12. Ooh ohh, was seh ich da...<div align="center">! Hab ich Dich zu früh gelobt? Raus damit! Trennung von Inhalt und Layout sollte schon gewährleistet sein!
13. Ohje! 51 Fehler meldet der Validator!  Ich habe Dich wirklich zu früh gelobt (Nun gut, der Code zeigt aber, dass Du um moderne Techniken bemüht bist, und das allein ist schon lobenswert (naja, eigentlich finde ich das selbstverständlich ;) ))
14. <br /> - Wozu das? Sieht nach Layout aus. Raus damit!
15. JavaScript, das ein Image auf die Seite bringt? Ich bin jetzt zu faul, JS abzuschalten um zu sehen, was passiert. Aber das ist sicher nichts wichtiges, oder? Dann sollte das nämlich raus!
16. Ohje! Der Tabellenwächter(TM) entdeckt beim Newsletter-Formular eine Layout-Tabelle! Pfui! Raus damit!
17. Semantik: Die Überschrift der Site und die Seitenüberschrift haben die selbe Ordnung (<h1>)? Für mich macht das keinen Sinn, die Überschrift der Site ("aktionsgutschein.de ...") sollte <h1> bekommen, die Seitenüberschrift (hier: "Impressum") maximal <h2>.
18. Mir fällt gerade ein, dass ich mal was gelesen habe, dass Websites, die die Domain in ihrem Namen haben und damit werben (wie bei Dir in der ersten Überschrift), als gewerbliche Sites gelten. Ach so...moment, Du hast ja ne UStID-Nr. Dann lass mich das anders formulieren: Fehlt da die Unternehmensform im Impressum? Oder ist das ne Einzelunternehmung?
Ich denk das reicht fürs erste
Bei Fragen bitte einfach rückfragen, auch wenn Verbesserungsvorschläge nötig sein sollten. Bei meinen Antworten muss ich ohnehin schon Angst haben, dass das Forum das nicht annimmt aufgrund der Länge)  |
| von Herr_Zatacke |
mir grad noch aufgefallen... : die 1x1Pixel-Grafik funktioniert garnicht...
1: | http://banners.webmasterplan.com/view.asp?site=2862&ref=231852&b=0&type=text&tnb=1 |
|
| von Efchen |
crazyhero schrieb am 17.06.2005 10:38
Hallo. das <p /> dachte ich immer steht für einen Zeilenumbruch. Deshalb habe ich es alf Formatierung eingesetzt. Ich werde versuchen das zu ändern. |
<p /> gibt es nicht. Es handelt sich dabei um ein Tag, das schon immer geöffnet und geschlossen werden sollte und nicht inhaltsleer ist. Nur inhaltsleere Tags, wie <img> schreibt man mit abschließendem Slash im selben Tag.
<p> steht außerdem nicht für Zeilenumbruch, sondern für "Textabsatz". HTML ist eine Strukturbeschribungssprache. "Zeilenumbruch" ist Layout, und sowas gehört nur ins CSS.
Ein Zeilenumbruch ist <br />, aber selbst da gibt es Leute, die sagen, dass das Tag keine Existenzberechtigung hat. Einen Zeilenumbruch kann man auch mit einem neuen Absatz ohne margin erreichen (nicht ganz meine Meinung). Wer aber mehr als ein <br /> braucht, will eigentlich einen Abstand.
Einen Abstand macht man jedenfalls immer mit margin. |
|
