KurtM Skrevet 7. februar 2006 Rapporter Del Skrevet 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! Lenke til kommentar
resol Skrevet 7. februar 2006 Rapporter Del Skrevet 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! Lenke til kommentar
KurtM Skrevet 8. februar 2006 Forfatter Rapporter Del Skrevet 8. februar 2006 Thx Hehe, har fixa title nå.. Lenke til kommentar
Gilbert Skrevet 9. februar 2006 Rapporter Del Skrevet 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! Lenke til kommentar
MindTooth Skrevet 9. februar 2006 Rapporter Del Skrevet 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 Lenke til kommentar
neivoll Skrevet 9. februar 2006 Rapporter Del Skrevet 9. februar 2006 (endret) 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 Lenke til kommentar
KurtM Skrevet 9. februar 2006 Forfatter Rapporter Del Skrevet 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.. Lenke til kommentar
Lovskogen Skrevet 9. februar 2006 Rapporter Del Skrevet 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. Lenke til kommentar
MrBjoern Skrevet 12. februar 2006 Rapporter Del Skrevet 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 Lenke til kommentar
Anbefalte innlegg
Opprett en konto eller logg inn for å kommentere
Du må være et medlem for å kunne skrive en kommentar
Opprett konto
Det er enkelt å melde seg inn for å starte en ny konto!
Start en kontoLogg inn
Har du allerede en konto? Logg inn her.
Logg inn nå