Fahrradhaus

[t-mow]

Hallo,

wollte mal mein erstes ZF-Projekt, welches seit kurzem online ist, präsentieren:
www.fahrradhaus-schawo.de (http://www.fahrradhaus-schawo.de)

Ist größtenteils nur eine statische "Visitenkarte", allerdings gibt es auch dynamische Teile, wie z.B. eine Artikelverwaltung, etc...

Unter anderem kommen folgende Komponenten vor:
Komplettes MVC-Konzept
Zend_Acl & Zend_Auth für den Administrationsbereich
Zend_Config
Zend_Layout
Zend_Db_* für die Models
Zend_Form für das Kontaktformular
Zend_Search_Lucene für die SucheFreue mich auf eure Kommentare :)

[deetee]

Hi,

alles in allem gefällt mir das gut. Die Farbkombi Gelb und Rot gefällt mir persönlich nicht so. Was ich ändern würde sind auf jedenfall die URLs. Das "index" finde ich weniger prickelnd. Design technisch sieht alles sehr sauber aus. Gefällt mir.

edit:
Ach ja, nochmal zu den URLs. Sowas hier fahrradhaus-schawo.de/produkte/kategorie/id/2 kann man auch sehr einfach "sprechender" und "einprägender" umschreiben mit den Router Klassen.

[DennisBecker]

Sieht ganz ok aus von den Farben her ;) die URL's /index/* find ich auch unschön. Negativ aufgefallen ist mir auch /kategorie/id/3 - warum hängst du den Namen nicht hintendran? Das ist auch gut für's Ranking bei Suchmaschinen.

Zudem liefert http://www.fahrradhaus-schawo.de/produkte/kategorie/id/4 eine falsche Fehlermeldung ;)

Ansonsten sieht es echt gut aus. Mir fehlen irgendwie noch Begiffe bei der Suche. Wenn man "Helm" eingibt, gibt es keine Treffer. "Shimano" ebenfalls nicht. Wird da nur der Produktname indexiert?

[Kaiuwe]

Thema HTML:

leere Absätze, leere Listenpunkte und Zeilenumbrüche als Abstandhalter
Überschriften und Listen als Absatz
Tabellendaten als Absätze
Adressdaten als Bild
verschachtelte Listen nicht ausgezeichnet
...


Wirklich fett ist die Startseite in Kilobyte: "Total 172 KB (189 KB uncompressed)"

[t-mow]

Zunächst mal danke für die Kritik. Die index/* Problematik werde ich dann wohl über eine extra Route(?) beheben.
@Radhad (:P) Die Fehlemeldung bei der Kategorie habe ich mal abgeändert ;)

Thema HTML:
leere Absätze, leere Listenpunkte und Zeilenumbrüche als Abstandhalter
Überschriften und Listen als Absatz
Tabellendaten als Absätze
Adressdaten als Bild
verschachtelte Listen nicht ausgezeichnet
...Wirklich fett ist die Startseite in Kilobyte: "Total 172 KB (189 KB uncompressed)"

Ein wenig zu meiner Verteidigung:

Überschriften und Listen als Absatz: wenn man Listen und Überschriften per CSS nicht selbst styled, haben diese doch auch einen "default" Abstand... Also eine H1 z.B. mehr als eine H2, etc...
Leere Absätze: ich vermute, da meinst du die Absätze in der Produktbeschreibung. Diese resultieren aus einem WYSIWYG Editor. Wenn da bei der Eingabe der Produkte also irgendwie was falsch gemacht wird, ists natürlich doof :(

[Kaiuwe]

Zunächst mal danke für die Kritik. Die index/* Problematik werde ich dann wohl über eine extra Route(?) beheben.
Ja!

Überschriften und Listen als Absatz: wenn man Listen und Überschriften per CSS nicht selbst styled, haben diese doch auch einen "default" Abstand... Also eine H1 z.B. mehr als eine H2, etc...
Es werden Überschriften und Listen als Absätze ("p" - "paragraph") ausgegeben und nicht als das was sie eigentlich sind! CSS ist da uninteressant.

Ergänzungen:

für die Linkliste (http://www.fahrradhaus-schawo.de/index/links) wäre eine Definitionsliste besser
der gesamte Adressteil im Fuß der Seite (http://www.fahrradhaus-schawo.de/) ist ein Bild
die Navigationsliste (http://www.fahrradhaus-schawo.de/produkte) beinhaltet eigentlich eine verschachtelte Liste, diese ist aber nicht gekennzeichnet
(tabellarische) Produktdaten (http://www.fahrradhaus-schawo.de/produkte/details/id/108) werden per Absatz und stark betont gekennzeichnet
leerer Listenpunkte als "Spacer" in der Navigation (http://www.fahrradhaus-schawo.de/)

[Kaiuwe]

Da wir ja in einem Zend Framework-Forum sind: ;)

Dem Kontaktforumlar würde ich noch ein paar Prüfungen hinzufügen: (z.B.)

Mindestlänge ebenfalls für Wohnort, Telefon und Anfragefeld
Betreff per InArray (http://framework.zend.com/manual/de/zend.validate.set.html#zend.validate.set.in_array) prüfen

...und filtern (http://framework.zend.com/manual/de/zend.form.elements.html#zend.form.elements.filters ) nicht vergessen!

[t-mow]

Die Definitionsliste bei den Links hab ich schon mal eingebaut, sieht in der Tat besser aus ;)

Die URLs habe ich auch weitestgehend optimiert:

$router = $frontController->getRouter();
$router->addRoute('produkt', new Zend_Controller_Router_Route('produkte/details/:id', array('controller'=>'produkte','action'=>'details','id'=>'')));
$router->addRoute('indexFix', new Zend_Controller_Router_Route(':action', array('controller'=>'index','action'=>'')));
Werde mich die Tage an die restlichen Sachen machen - danke an alle ;) (weitere Kommentare sind natürlich trotzdem erlaubt :P)

[mutwalek]

Mmmh, wenn man im Kontaktformular eine falsche EMail Adresse eingibt, kommt eine Fehlermeldung auf Englisch, was natürlich für eine deutsche Seite nicht so schön ist.

Aber sonst schön gelungen.

Gruß
Jürgen Mutwalek