Gå til innhold

Trenger Konstruktiv kritikk

Tafsen

Av Tafsen
i Programmering og webutvikling

Anbefalte innlegg

Videoannonse

Videoannonse
Annonse
resol

resol

Skrevet
Skrevet

Utseende messig gjore den jo nytta si, innholdet er i fokus og det er det som er viktig, får ikke med en gang inntrykk av at dette er et byggeservicefirma, men tipper de som går inn der alt vet hva de går til.. Greie farger også.

 

Kodemessig var det svakt, ikke doctype, og i det hele tatt ganske labert koda.

Du burde kanskje se litt på css?

 

forøvring var det en del snåle ting i koden, her er en liste ( med hilsen fra tidy i firefox)

 

line 1 column 1 - Warning: missing <!DOCTYPE> declaration

line 85 column 7 - Warning: discarding unexpected <html>

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

line 78 column 5 - Warning: <title> isn't allowed in <div> elements

line 78 column 5 - Warning: <meta> isn't allowed in <div> elements

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

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

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

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

line 120 column 10 - Warning: <title> isn't allowed in <div> elements

line 120 column 10 - Warning: <meta> isn't allowed in <div> elements

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

line 149 column 7 - Warning: discarding unexpected <html>

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

line 142 column 5 - Warning: <title> isn't allowed in <div> elements

line 142 column 5 - Warning: <meta> isn't allowed in <div> elements

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

line 223 column 1 - Warning: discarding unexpected </div>

line 38 column 1 - Warning: <body> proprietary attribute "leftmargin"

line 38 column 1 - Warning: <body> proprietary attribute "topmargin"

line 38 column 1 - Warning: <body> proprietary attribute "marginwidth"

line 38 column 1 - Warning: <body> proprietary attribute "marginheight"

Lenke til kommentar
Tafsen

Tafsen

Skrevet
  • Forfatter
Skrevet (endret)

:blush: *kremt*

ialfall ei av sidene er berre *host* konventert i Word :blush:

Så der ligger nok litt av årsakene til nokre av feilmeldingane :p

 

Kodinga er elendig, eg veit :no: Men tenkte eg skulle berre få sida nokon lunde ferdig så den kunne brukast, så får eg heller ta en skikelig oppvask i kodane etter på.

 

Eg bruker css, sjølv om eg kanskje ikkje har så masse peiling på det :roll:

 

Takk for svar :thumbup:

Endret av rEaliTY
Lenke til kommentar
Loomy

Loomy

Skrevet
Skrevet

  • For så vidt et OK utseende. Det gjør jobben, men er veldig lite kreativt. Mangler liksom det lille ekstra
  • Skriveleif i <title>. Velkommen skrives med to m'er ;)
  • Det heter "firmainfo", ikke "firma info".
  • Koden var rimelig grell, mangler først og fremst DOCTYPE
  • XHTML-avslutning på link-elementet i head
  • Masse råtne standard Dreamweaver-genererte javascript preloadere
  • Masse utdaterte attributter i body-tagen. Bør flyttes til CSS
  • Logoen bør legges som bakgrunnsbilde f.eks i en <h1> som også inneholder navnet på bedriften - istedenfor bare en img-tag som attpåtil mangler alt-tekst
  • Hvorfor i alle dager bruker du bilder i menyen? Vanlig tekst med CSS-hover-effekter ville vært en mye enklere løsning
  • Nå gidder jeg ikke kommentere koden mer, for den var horribel :p54 feil i validatorn sier det meste..

Lenke til kommentar
Tafsen

Tafsen

Skrevet
  • Forfatter
Skrevet
[*]Skriveleif i <title>. Velkommen skrives med to m'er ;)

 

 

[*]Hvorfor i alle dager bruker du bilder i menyen? Vanlig tekst med CSS-hover-effekter ville vært en mye enklere løsning

1: På nynorsk heite det Velkomen med ein m.

 

2: Fordi eg ville bruke ei font som ikkje er standard

Lenke til kommentar
Tafsen

Tafsen

Skrevet
  • Forfatter
Skrevet (endret)
[*]Masse råtne standard Dreamweaver-genererte javascript preloadere

Takker for tilbakemeldinga.

 

Men ka skal eg bruke i staden for den då?

 

Sidan eg bruker php, så skal vel eg bruke DOCTYPE declaration for php. Kor finner eg et slikt oppsett?

Endret av rEaliTY
Lenke til kommentar
Loomy

Loomy

Skrevet
Skrevet
Sidan eg bruker php, så skal vel eg bruke DOCTYPE declaration for php. Kor finner eg et slikt oppsett?

Nope, nettleseren aner ikke om du bruker PHP eller ikke (og kunne ikke brydd seg mindre ;)), så du skal bruke en helt vanlig HTML 4.01 DOCTYPE:

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

Lenke til kommentar
  • 1 måned senere...
Tafsen

Tafsen

Skrevet
  • Forfatter
Skrevet
En skikkelig imagereplacement metode :)

Google it.

4961672[/snapback]

Sida eg ikkje visste at min imagereplacement metode var dårlig så blir det vel litt vanskelig for meg å vete kva som er bra av det som eg finner på google.

 

Derfor lurte eg på om nokon kunne være så snill å anbefale ein metode til meg?

Lenke til kommentar
PT

PT

Skrevet
Skrevet
Hvorfor i alle dager bruker du image-rollover til ren-tekst? Og i det hele tatt masse bilder som bare er tekst? Fullstendig unødvendig!

 

Du kan få mye penere menyer med bare tekst og CSS.

5156174[/snapback]

Greit å lese hele tråden før du svarer :)

2: Fordi eg ville bruke ei font som ikkje er standard

4961431[/snapback]

Lenke til kommentar

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 konto

Logg inn

Har du allerede en konto? Logg inn her.

Logg inn nå
Gå til emneliste
×
×
  • Opprett ny...