Feedback?

07/19/2012 01:39 __BuRn3R#1
Hi Leute, ich bin derzeit dabei die Homepage meines Grafik Shops umzucoden, und wollte mal fragen, wie ihr sie bis jetzt findet: [Only registered and activated users can see links. Click Here To Register...]

Bis jetzt ist leider erst die Startseite fertig (fast). Ich weiß, der Code ist sehr unsauber, aber bis jetzt reichen meine Kentnisse für die Dinge, die ich benötige.

Bitte schreibt, was ich noch verbessern kann. Danke, Mfg

P.S. Das Design sieht original so aus: [Only registered and activated users can see links. Click Here To Register...]

Ich habs leider nicht geschafft, sie 1:1 nachzumachen :x

Wenn ihr wollt, könnt ihr mein Code ruhig korrigieren
07/19/2012 13:21 .Acu³#2
Finde ich garnicht schlecht. (Endlich mal jemand der was ordentliches präsentiert.. ;))
Vorallem die Textur im Header Liebe ich einfach :D

Kritikpunkte hab ich trotzdem;) Ich würde die "weiter" und "zurück" Buttons des Sliders rechts und links hinmachen und nicht zentriert. wirkt finde ich schöner.

Bei dem Menü würde ich dir zu einem "Lavalamp" Menü raten, schaus dir einfach an:
(Das zweite bei "Bonus")
[Only registered and activated users can see links. Click Here To Register...]
Also vom Prinzip her, dass der schwarze BG nach rechts mitslidet.

Ansonsten, gute Arbeit.


Noch was zum Logo: Der Obere Rand wirkt unscharf, kann sein dass das gewollt ist, finde aber das sieht nicht ganz so gut aus.

Und wenn du einen Coder suchst, der mit dir zusammenarbeiten, meld dich mal bei mir :>
07/19/2012 14:36 __BuRn3R#3
Joa, das mit der Navi hatte ich mir auch schon überlegt, jedoch hatte ich dafür keine js/jquery Kentnisse, danke für den Link und die Kritik. Das mit dem Slider werd ich auch noch ändern
07/19/2012 22:46 kissein#4
Ich bin mal so frei und korrigiere ein paar HTML Fehler.

HTML Code:
<div id="logo"><a href="#"><img src="image/logo.png" width="258" height="77" alt="Graphics Solution" /></a></div>
w3c valide und logo.png kann später für eigenes clienttracking benutzt werden, siehe z.b. youtube

Code:
<div id="scanline"></div>
<div id="abstandhalter"></div>
id="scanline" und id="abstandhalter" müssen als class deklariert werden, wenn sie nicht unique sein sollen

Code:
 Line 87, Column 357: end tag for element "p" which is not open
</p> in </div> ändern da mit <div id="footer_left"> geöffnet wurde

Das Trackingscript innerhalb des <body></body> am unteren ende platzieren.

und naja der klassiker, </html> fehlt ;)

------------------------------------------------------------------------

Zum Css gesagt, habs nur kurz überflogen aber z.b.

style.css:149
#infotext a { .... text-shadow: 1px 1px 0px white;}

style.css:23
a { .... text-shadow: 1px 1px 0px black;}

Die beiden heben sich gegenseitig auf und sichtbar ist nur der color:value von #idselektor infotext a


Ansonsten fleissig Entwicklerkonsolen von den gängigen Browsern benutzen.
Wenn die Grafiken selber erstellt werden, am besten gleich alles auf ein sheet packen und via clip koordinaten das jeweilige bild holen.
08/11/2012 15:16 __BuRn3R#5
Hab jetzt mal diese Flüchtigkeitsfehler gefixxt.

Auch wenn ich es sehr unflexibel geschrieben habe, ich werde den Code jetzt erstmal als Übergang nehmen, und ihn später nochmal anständig schreiben, und wie du bereits erwähnt hast alle Grafiken in einem Bild speichern.
__

Könnte mir derweil evtl. jemand eine index.php erstellen, dass hinter den Seiten beispielweise index.php?s=xy stehen würde? Währe super nett, mfg. Kritik könnt ihr weiter schreiben :)