Trenger tips/konstruktiv kritikk :)

KurtM · 7. februar 2006

Hei!

Holder på å gjøre ferdig http://www.beijing-08.com. Hadde vært fint med noe konstruktiv kritikk og gode tips! Husk at det enda er WIP

Kom gjerne med kommentarer på koden i forhold til SEO, og hvordan denne kan forbedres

På forhånd takk!

resol · 7. februar 2006

Ting du mangler ( syns jeg )

En tittel...altså skrive noe i <title></title>.

Ser ikke så veldig bra ut med "untitled dockument"

En hover / mouseover funksjon på menyer / linker

ellers greit nok design...

Foruten det så fortalte Tidy i firefox meg dette:

line 7 column 1 - Error: <sohu_cms_include:en_hp_style> is not recognized!

line 7 column 1 - Warning: discarding unexpected <sohu_cms_include:en_hp_style>

line 206 column 3 - Warning: <input> element not empty or not closed

line 207 column 3 - Warning: <input> element not empty or not closed

line 209 column 3 - Warning: <input> element not empty or not closed

line 222 column 29 - Warning: discarding unexpected <!DOCTYPE>

line 223 column 1 - Warning: discarding unexpected <html>

line 224 column 1 - Warning: discarding unexpected <head>

line 222 column 25 - Warning: <meta> isn't allowed in <td> elements

line 222 column 25 - Warning: <title> isn't allowed in <td> elements

line 229 column 1 - Warning: discarding unexpected <body>

line 234 column 60 - Warning: discarding unexpected </span>

line 7 column 31 - Warning: <style> inserting "type" attribute

line 49 column 13 - Warning: <td> proprietary attribute "background"

line 52 column 13 - Warning: <td> proprietary attribute "background"

line 56 column 5 - Warning: <div> proprietary attribute "state"

line 57 column 2 - Warning: <div> proprietary attribute "classout"

line 57 column 2 - Warning: <div> proprietary attribute "classover"

line 59 column 2 - Warning: <div> proprietary attribute "state"

line 59 column 34 - Warning: <span> proprietary attribute "classout"

line 59 column 34 - Warning: <span> proprietary attribute "classover"

line 60 column 2 - Warning: <span> proprietary attribute "classout"

line 60 column 2 - Warning: <span> proprietary attribute "classover"

line 61 column 2 - Warning: <span> proprietary attribute "classout"

line 61 column 2 - Warning: <span> proprietary attribute "classover"

line 66 column 17 - Warning: <div> proprietary attribute "state"

line 67 column 2 - Warning: <div> proprietary attribute "classout"

line 67 column 2 - Warning: <div> proprietary attribute "classover"

line 69 column 2 - Warning: <div> proprietary attribute "state"

line 69 column 34 - Warning: <span> proprietary attribute "classout"

line 69 column 34 - Warning: <span> proprietary attribute "classover"

line 70 column 2 - Warning: <span> proprietary attribute "classout"

line 70 column 2 - Warning: <span> proprietary attribute "classover"

line 71 column 2 - Warning: <span> proprietary attribute "classout"

line 71 column 2 - Warning: <span> proprietary attribute "classover"

line 76 column 5 - Warning: <div> proprietary attribute "state"

line 77 column 2 - Warning: <div> proprietary attribute "classout"

line 77 column 2 - Warning: <div> proprietary attribute "classover"

line 79 column 2 - Warning: <div> proprietary attribute "state"

line 79 column 34 - Warning: <span> proprietary attribute "classout"

line 79 column 34 - Warning: <span> proprietary attribute "classover"

line 80 column 2 - Warning: <span> proprietary attribute "classout"

line 80 column 2 - Warning: <span> proprietary attribute "classover"

line 81 column 2 - Warning: <span> proprietary attribute "classout"

line 81 column 2 - Warning: <span> proprietary attribute "classover"

line 86 column 5 - Warning: <div> proprietary attribute "state"

line 87 column 2 - Warning: <div> proprietary attribute "classout"

line 87 column 2 - Warning: <div> proprietary attribute "classover"

line 89 column 2 - Warning: <div> proprietary attribute "state"

line 89 column 34 - Warning: <span> proprietary attribute "classout"

line 89 column 34 - Warning: <span> proprietary attribute "classover"

line 90 column 2 - Warning: <span> proprietary attribute "classout"

line 90 column 2 - Warning: <span> proprietary attribute "classover"

line 91 column 2 - Warning: <span> proprietary attribute "classout"

line 91 column 2 - Warning: <span> proprietary attribute "classover"

line 96 column 5 - Warning: <div> proprietary attribute "state"

line 97 column 2 - Warning: <div> proprietary attribute "classout"

line 97 column 2 - Warning: <div> proprietary attribute "classover"

line 99 column 2 - Warning: <div> proprietary attribute "state"

line 99 column 34 - Warning: <span> proprietary attribute "classout"

line 99 column 34 - Warning: <span> proprietary attribute "classover"

line 100 column 2 - Warning: <span> proprietary attribute "classout"

line 100 column 2 - Warning: <span> proprietary attribute "classover"

line 101 column 2 - Warning: <span> proprietary attribute "classout"

line 101 column 2 - Warning: <span> proprietary attribute "classover"

line 106 column 5 - Warning: <div> proprietary attribute "state"

line 107 column 2 - Warning: <div> proprietary attribute "classout"

line 107 column 2 - Warning: <div> proprietary attribute "classover"

line 109 column 2 - Warning: <div> proprietary attribute "state"

line 109 column 34 - Warning: <span> proprietary attribute "classout"

line 109 column 34 - Warning: <span> proprietary attribute "classover"

line 110 column 2 - Warning: <span> proprietary attribute "classout"

line 110 column 2 - Warning: <span> proprietary attribute "classover"

line 111 column 2 - Warning: <span> proprietary attribute "classout"

line 111 column 2 - Warning: <span> proprietary attribute "classover"

line 116 column 5 - Warning: <div> proprietary attribute "state"

line 117 column 2 - Warning: <div> proprietary attribute "classout"

line 117 column 2 - Warning: <div> proprietary attribute "classover"

line 119 column 2 - Warning: <div> proprietary attribute "state"

line 119 column 34 - Warning: <span> proprietary attribute "classout"

line 119 column 34 - Warning: <span> proprietary attribute "classover"

line 120 column 2 - Warning: <span> proprietary attribute "classout"

line 120 column 2 - Warning: <span> proprietary attribute "classover"

line 121 column 2 - Warning: <span> proprietary attribute "classout"

line 121 column 2 - Warning: <span> proprietary attribute "classover"

line 126 column 5 - Warning: <div> proprietary attribute "state"

line 127 column 2 - Warning: <div> proprietary attribute "classout"

line 127 column 2 - Warning: <div> proprietary attribute "classover"

line 129 column 2 - Warning: <div> proprietary attribute "state"

line 129 column 34 - Warning: <span> proprietary attribute "classout"

line 129 column 34 - Warning: <span> proprietary attribute "classover"

line 130 column 2 - Warning: <span> proprietary attribute "classout"

line 130 column 2 - Warning: <span> proprietary attribute "classover"

line 131 column 2 - Warning: <span> proprietary attribute "classout"

line 131 column 2 - Warning: <span> proprietary attribute "classover"

line 136 column 5 - Warning: <div> proprietary attribute "state"

line 137 column 2 - Warning: <div> proprietary attribute "classout"

line 137 column 2 - Warning: <div> proprietary attribute "classover"

line 139 column 2 - Warning: <div> proprietary attribute "state"

line 139 column 34 - Warning: <span> proprietary attribute "classout"

line 139 column 34 - Warning: <span> proprietary attribute "classover"

line 140 column 2 - Warning: <span> proprietary attribute "classout"

line 140 column 2 - Warning: <span> proprietary attribute "classover"

line 141 column 2 - Warning: <span> proprietary attribute "classout"

line 141 column 2 - Warning: <span> proprietary attribute "classover"

line 186 column 13 - Warning: <td> proprietary attribute "background"

line 189 column 19 - Warning: <td> proprietary attribute "background"

line 193 column 21 - Warning: <iframe> escaping malformed URI reference

line 193 column 21 - Warning: <iframe> attribute "align" has invalid value "top"

line 193 column 21 - Warning: <iframe> attribute value "No" must be lower case for XHTML

line 197 column 19 - Warning: <td> proprietary attribute "background"

line 216 column 19 - Warning: <td> proprietary attribute "background"

1 error / 111 warnings

Det er ting du kan se på

lykke til!

KurtM · 8. februar 2006

Thx Hehe, har fixa title nå..

Gilbert · 9. februar 2006

Fine farger! Har allerde skrevet to lengre kommentarer på sider, så her blir det vel litt stikkord:

Altså, fine farger! Du holder deg til én farge, men blandet med den røde i flagget og den gule hvitfargen på bakgrunnen på bildet i midten er kombinasjonen nydelig
Lite margin over og under overskrifter
Får følelsen av at du har kompansert den lille skriftstørrelsen "Placeholder" med å gjøre den bold, i menyen altså.
Bakgrunnen på "newsletter" content er anderledes enn de andre boksene på høyre side
Bildene under menyen til venstre er kornete
Siden er svært behagelig å lese og se på generelt. Topp!

MindTooth · 9. februar 2006

Ta en titt på denne:

http://validator.w3.org/check?uri=http%3A%...&doctype=Inline

Ellers synes jeg de orange knappene ser litt old fashin ut, og trenger litt gloss

neivoll · 9. februar 2006

Designet så jo stort sett veldig bra ut. Det som kanskje burde gjøres noe med er knappene på menyen. Folk har allerede nevnt hover-effekt og "gloss"(!), og du bør iallefall få midtstilt teksten vertikalt, den er trykket litt opp i rammen.

edit: legg også merke til at alle knapper/oransje felter slutter veldig brått, i forhold til de grå feltene som slutter med en skygge-effekt.

Endret 9. februar 2006 av anners

KurtM · 9. februar 2006

Ærh, det er noe med siden jeg ikke er helt fornøyd med.. Men klarer ikke helt å sette fingern på det.. Lurer på om det er noe med fargene.. hmm..

Lovskogen · 9. februar 2006

Ikke 'box' sammen layouten, altså, fjern de gråe linjene
Litt tacky effekter i headern, keep it simple
Snakker om tacky, menyen din. Bruk gjerne de fine fargene, men gjør den litt mer livlig, og skaff deg en hover.
'Sponson'-linkene burde gå nedover, ikke samle alle i en klase
Du burde style img med litt padding og en border.
Footern virka litt unødvendig

Skal komme med flere tilbakemeldinger etter at du har fått spist opp det jeg nettopp skrev.

MrBjoern · 12. februar 2006

Ser bra ut det der.

Eneste jeg la merke til er at linkene ikke virker (gjør ivertfal ikke det hos meg)

Ellers var fargene ok og siden va stilrein.

Går ut ifra at det står placeholder på nesten alle linkene fordi siden ikke er ferddig enda hehe

Logg inn

Trenger tips/konstruktiv kritikk :)

Anbefalte innlegg

KurtM

Lenke til kommentar

Videoannonse

resol

Lenke til kommentar

KurtM

Lenke til kommentar

Gilbert

Lenke til kommentar

MindTooth

Lenke til kommentar

neivoll

Lenke til kommentar

KurtM

Lenke til kommentar

Lovskogen

Lenke til kommentar

MrBjoern

Lenke til kommentar

Opprett en konto eller logg inn for å kommentere

Opprett konto

Logg inn

Hvem er aktive 0 medlemmer