Mag jemand versiertes meinen code beurteilen?

[rockscientist01]

Hi Leute,

ich bin relativ neu im webdesign, und ich habe kaum Möglichkeiten, mich mit erfahrenen Webdesignern auszutauschen.
Jetzt habe ich eine Seite fertig gemacht, von der ich denke, dass sie ganz gut gelungen ist. Dem Kunden gefällt sie jedenfalls sehr.
Die Sache ist: Mich würde mal interessieren, was ihr erfahrenen profis vom code haltet... Kann ich das so machen, oder habe ich da no-gos drin? Also ich hätte einfach gerne mal ein bisschen feedback, vielleicht könnt ihr mir helfen ?!?!

FunkyFlair - Home

Vielen Dank!
AO

 

[threadi]

Moderation: Verschoben von HTML zu ShowCase.

 

[rockscientist01]

sorry! Werd nächste mal auf die Kategorie achten!

 

[Wustersoss]

Hallo,

auf den ersten Blick sehr schönes Design. Insbesondere das Logo und die Spielerei mit Schwarzweiss und Rot in den Bildern finde ich gut gemacht

Auf den zweiten Blick fallen das Menü auf. Die Worte passen nicht in die Tabs.
Der Text auf der Startseite ist Blocksatz und enthält grosse Lücken sowie ein Trennungsstrich bei Trauungsze- remonien.
Bei den Linkempfehlungen sollten auch wirklich nur Links drin stehen und nicht E-Mail-Adressen, die dann mein Outlook öffnen, sehr ärgerlich.
Auf allen Seiten muss der Text noch mal revidiert werden. Es sind viele Worte mit Trennungszeichen und Fehlern drin, ein Wunder, dass die Band das so abgenommen hat.
Auf der Seite Band zum Beispiel: Vovalcoach (soll wohl Vocalcoach heissen)
Die Schrift finde ich nicht gut gewählt, zu gross, zu fett, schlecht lesbar für mich.


Was ich als Kunde vermissen würde:

Wo treten sie demnächst auf um sie mal live zu sehen?
Warum ist kein Video von einem Auftritt auf der Seite?
Warum kein Buchungskalender um sofort zu sehen wann sie ausgebucht sind.
Wie kann ich sie buchen und warum ist dort nirgends ein Preis zu finden?

Ein Gästebuch für Feedbacks fände ich auf so einer Seite auch nicht schlecht.

 

[Anusha]

Ueber das Design kann/will ich nichts sagen.
Dein Quellcode jedenfalls schaut gut aus. Ist gut strukturiert und vor allem valide.
Weiter so ;)

 

[vitus37]

Der Code ist schwer in Ordnung. Würde nur die Meta-Description etwas mehr ausführen (irgendetwas negatives muss ich ja sagen :mrgreen:).

 

[rockscientist01]

Hallo Leute,

also erst mal vielen Dank für euer feedback.- Ohne feedback fühle ich mich immer so verloren, weil ich nicht weiß, ob das jetzt gut so ist oder eben nicht, was ich da fabriziert hab...

@Wustersoss:
Danke für Dein Lob bezüglich des Designs.
Deiner weiteren Beschreibung zufolge scheint die Seite bei Dir anders auszusehen, als bei mir im FF, Ie8&9, und Chrome. Ich hab mal einen screenshot angehängt, wie sie bei mir aussieht. Da hat der Trennstrich bei Trauungsze- remonien nämlich durchaus seinen Sinn. Bei mir passen die Wörter auch gut in die Menütabs. Vielleicht kannst du auch mal einen Screenshot machen?
Um Rechtschreibung habe ich mich wirklich nicht gekümmert: Wenn mir was aufgefallen war, dann habe ich das korrigiert, aber ich bin nicht jeden Text konzentriert durchgegangen, dafür hab ich ja euch - ne war natürlich nur Spaß! Das Korrigieren, denke ich, sollten die Verfasser der Texte machen, in diesem Fall die Band (Nichtsdestotrotz habe ich Vocalcoach mal eben verbessert - danke dafür!). Zu den anderen Punkten: Es ist natürlich immer gut, wenn man noch mehr und dann noch mehr in die webseite einbaut. Warum nicht gleich eine Buchungsformular mit Bezahlmöglichkeit? Oder eine automatisierte Möglichkeit, ein Lied herunterzuladen, in dem man den Namen von Braut und Bräutigam oder Geburtstagskind hört? Das sind alles Inhaltliche Fragen, die zu diskutieren es nicht lohnt, weil der Inhalt vom Kunden nunmal so gewünscht, abgenommen und bezahlt wurde.
Mich hat nur mal für mich konkret interessiert, wie andere web erfahrene meinen code beurteilen. Und was ich da so rausgehört habe, scheint er im Groben ok zu sein. Das macht mir Mut, weiter zu machen :)
Vielen Dank für Dein feedback!

@Anusha:
Danke für Dein feedback und Lob!

@vitus37:
Danke für Dein Lob! Dass Du die meta-tags erwähnst is lustig: Ich lese nämlich gerade über meta tags, weil mir aufgefallen ist, dass viele Webseiten einen riesen Wust an meta-tags im head haben. Und ich wusste gar nichts damit anzufangen. Aber langsam steige ich dahinter... Danke für den Tip und Dein feedback!


liebe Grüße
AO

 

[vitus37]

Dass Du die meta-tags erwähnst is lustig: Ich lese nämlich gerade über meta tags, weil mir aufgefallen ist, dass viele Webseiten einen riesen Wust an meta-tags im head haben. Und ich wusste gar nichts damit anzufangen.

Die meisten Meta-Tags halte ich eigentlich für unnötig. Da Google jedoch oftmals den Inhalt des Description-Tags in seine Ergebnisbeschreibungen aufnimmt und ein paar lose Keywords dort nicht so schön aussehen, würde ich den Inhalt der Meta-Description ausformulieren.

 

[harryman]

Hi,

nettes Design - sieht schön aus.

Ich würde versuchen, die Bilder noch ein wenig zu komprimieren, z.B. das gal_3.jpg mit > 500kB. Da ist noch einiges an "Luft" zum rauslassen drin.
Vorteil: Seitenaufbau - besonders beim Erstaufruf - geht deutlich schneller ==> da freut sich der User und die Suchmaschine

 

[rockscientist01]

ok guter Tip, danke!

 

[Wustersoss]

Bei mir sieht die Seite so aus. Ich habe mal geschaut, du verwendest die Schriftart Josefine-sans, die ich nicht habe.
Ansicht ist FF9 auf Windows XP:

 

[rockscientist01]

hmm komisch die Schrift hat glaub ich kaum jemand, deswegen ist sie auch über google-fonts eingebunden:

html:

<link href="styles/style.css" type="text/css" rel="stylesheet" />

CSS: 

@import url(http://fonts.googleapis.com/css?family=Josefin+Sans:100,300,400,600,700);

Wie sieht es denn bei euch anderen so aus? Wie bei mir, oder wie bei "Wustersoss"?
Ich hab jetzt schon mal "Arial" als Ersatzschrift eingerichtet, vlt hilft das schonmal.

 

[Wustersoss]

hm - das ist jetzt merkwürdig, weil dann müsste FF9 die doch auch nehmen. Chrome und Opera zeigen es richtig an.

Aha, meine Error-console zeigt mir das hier an:

Error: downloadable font: download not allowed (font-family: "Josefin Sans" style:normal weight:600 stretch:normal src index:2): content blocked
source: http://themes.googleusercontent.com...YlRF50SGJq1XyXj04z9Od4RnLmtUrdSQkDmC5b8k.woff
Source File: http://fonts.googleapis.com/css?family=Josefin+Sans:100,300,400,600,700
Line: 0
Source Code:
@font-face { font-family: "Josefin Sans"; font-style: normal; font-weight: 600; src: local("Josefin Sans SemiBold"), local("JosefinSans-SemiBold"), url("http://themes.googleusercontent.com/static/fonts/josefinsans/v3/C6HYlRF50SGJq1XyXj04z9Od4RnLmtUrdSQkDmC5b8k.woff") format("woff"); }

Ich glaube das liegt daran, das ich Google geblockt habe im FF.

 

[rockscientist01]

jo daran wirds dann wohl liegen. Warum hast du "google" geblockt?

 

[Wustersoss]

Warum hast du "google" geblockt?

Nicht nur Google ist geblockt.

Darum -> Welche Daten sammelt Google, ein interessanter Überblick | Datenwachschutz

 

[xXxPeterPanxXx]

jo daran wirds dann wohl liegen. Warum hast du "google" geblockt?

Machen viele ;)

Auch ich blocke mit NoScript so viel wie möglich. Man surft einfach sicherer und anonymer.

 

[Anusha]

Hm ... ich hoffe ihr habt dann allle keinen Facebook Account ;)

Facebook auf fremden Sites | c't

Jede Seite, die Facebook Plugins beinhaltet sendet seine Daten an Facebook, selbst wenn man gar nicht eingeloggt ist...

 

[efwe]

sympatische seite. aber das: text-align: justify; -sieht recht hesslich aus. blocksatz ist nicht gut und die fette grosse schrift erschlaegt einen - die listenpunkte beim menue sollten sich dem text anpassen. ich vermisse hoerproben-oder hab ich die uebersehen?

 

[sysop]

Schön und sauber gemacht, die Seite funktioniert soweit ich sehen konnte auch problemlos ohne JS.
Google ist auch bei mir geblockt und NEIN, NATÜRLICH habe ich KEINEN Gesichtsbuch account, die sind ja noch schlimmer als Google und co.

Skalieren der Seite funktioniert gut und alles passt sich sauber ein. Das Tabs-Problem kann ich trotz Google-Blockade nicht nachvollziehen, bei mir passen die Texte in die Tabs, sehen vielleicht beim einen oder anderen Eintrag etwas hineingequetscht aus. Die Tabs dürfte man also etwas grösser machen, ansonsten...

Echt gelungen

 

[Körnerbrötchen]

Hey, nur ganz kurz:

[TABLE]
[TR]
[TD]<div id="gallery">[/TD]
[/TR]
[TR]
[TD][/TD]
[TD]    <a class="fancybox" rel="gallery1" href="[URL="http://www.funky-flair.de/images/24.JPG"]images/24.JPG[/URL]"><img src="[URL="http://www.funky-flair.de/images/24.png"]images/24.png[/URL]" alt="funky Flair als Gruppe in Frankfurt" /></a>[/TD]
[/TR]
[TR]
[TD][/TD]
[TD]    <a class="fancybox" rel="gallery1" href="[URL="http://www.funky-flair.de/images/81.JPG"]images/81.JPG[/URL]"><img src="[URL="http://www.funky-flair.de/images/81.png"]images/81.png[/URL]" alt="funky Flair als Gruppe in Frankfurt" /></a>[/TD]
[/TR]
[TR]
[TD][/TD]
[TD]    <a class="fancybox" rel="gallery1" href="[URL="http://www.funky-flair.de/images/64.JPG"]images/64.JPG[/URL]"><img src="[URL="http://www.funky-flair.de/images/64.png"]images/64.png[/URL]" alt="funky Flair als Gruppe in Frankfurt" /></a>[/TD]
[/TR]
[TR]
[TD][/TD]
[TD]    <a class="fancybox" rel="gallery1" href="[URL="http://www.funky-flair.de/images/101.JPG"]images/101.JPG[/URL]"><img src="[URL="http://www.funky-flair.de/images/101.png"]images/101.png[/URL]" alt="funky Flair als Gruppe in Frankfurt" /></a>[/TD]
[/TR]
[TR]
[TD][/TD]
[TD]    <a class="fancybox" rel="gallery1" href="[URL="http://www.funky-flair.de/images/46.JPG"]images/46.JPG[/URL]"><img src="[URL="http://www.funky-flair.de/images/46.png"]images/46.png[/URL]" alt="funky Flair als Gruppe in Frankfurt" /></a>[/TD]
[/TR]
[TR]
[TD][/TD]
[TD]    <a class="fancybox" rel="gallery1" href="[URL="http://www.funky-flair.de/images/67.JPG"]images/67.JPG[/URL]"><img src="[URL="http://www.funky-flair.de/images/67.png"]images/67.png[/URL]" alt="funky Flair als Gruppe in Frankfurt" /></a>[/TD]
[/TR]
[TR]
[TD]</div>[/TD]
[TD][/TD]
[/TR]
[/TABLE]

Semantisch korrekt müsste das eine Liste sein.