Må vel være enig med de andre her, det ser bra ut, ryddig og greit.
Har uansett et par tips!
Får begynne øverst, du resetter CSS'en, props for det, noe de fleste helt glemmer, men du kunne kuttet en forespørsel til serveren ved å sette inn den resetten øverst i stil.css i stedet for å bruke en egen fil.
Du linker inn favicon, og det er feil å sette en "a" foran "href", altså burde det vært:
HTML:
<link rel="shortcut icon" href="favicon.ico">
Du har et par steder både startet og avsluttet <br> tags, noe som ikke er mulig ettersom disse kun gir et linjebrudd, og du kan ikke bruke </br>, i HTML5 holder det med med kun <br> der hvor du ønsker å bryte teksten.
Du mangler <alt> på et par bilder, og du bør normalt sette størrelsen til bildene i CSS.
Du har også avsluttet et par paragrafer som aldri er påbegynt, altså du har noen </p> for mye !
I CSS filen din har du alt riktig nedover, før du plutselig har noen åpenlyse feil som helt garantert bare er skrivefeil ettersom du har dette riktig hele veien ellers,
feilene er :
HTML:
.omoss1 {
background-image: url(IMG/omoss_hover.png);
height: 10;
width: 52px;
float: left;
margin-top: 10px;
height: 20px;
background-repeat: no-repeat;
background-position: center center;
}
Du må angi px for height !
HTML:
.karttext {
height: 130px;
width: 400px;
margin-left: 130px;
font-family: Arial, Helvetica, sans-serif;
font-size: 12px;
color: #CCC;
}
}.kontaktbilde {
height: 124px;
width: 290px;
background-image: url(IMG/jan.png);
background-repeat: no-repeat;
background-position: left center;
margin-left: 80px;
}
Her har du vel avsluttet litt vel mye ??
Du har en del CSS som jeg ikke umiddelbart finner noe bruk for, og det ser ut som det er en del som har blitt liggende igjen som ikke lengre brukes, det er alltid litt vanskelig å se hvor alt hører hjemme så det er mulig jeg tar feil, men kode som ikke er til noe bør normalt fjernes.
På siden for tjenester fant jeg en pussig sak:
HTML:
<br>Vi har faste samarbeidspartnere innen:</br>
<p> <ul>
<b><li>Grunnarbeid</li>
<li>Vvs</li>
<li>Støping</li>
<li>Flis</li>
<li>Elektronikk</li>
<li>Maling</li>
<li>Blikkventilasjon</li></b>
</ul></p>
<br> og <p> er brukt på en kreativ måte, og hva gjør den <b> taggen der midt i en liste??
Noe merkelig på kontakt siden:
HTML:
<div class="kontakttext2">
<h2>Jan Wilmann</h2>
<h3><p>Byggmester</h3></p>
<br>Tlf: <b>905 96 706</b></br>
<br>E-post: <b>jan@jarabygg.no</b></br>
</div>
<br> igjen ???? Her kan du forsåvidt bruke <b> slik som du gjør, men det ville vært bedre å satt dette i CSS.
Og, du må enten velge h3 eller p, du kan ikke bruke begge ???
Dette er for det meste små feil som du retter raskt, og så validerer du siden din for å sjekke at alt er i orden, jeg forstår det slik at det har vært litt tidspress for å få siden på nett, men nå i ettertid er det sikkert litt rom for å få alt helt hundre.
Du har også gjort samme feil som nesten alle andre webdesignere og helt glemt å sette opp webhotellet / serveren.
Dersom du har mulighet benytter du gzip og setter "expires headers" og deaktiverer "Etags" i .htaccess.
Du kan også spare en del på minimere css, javascript og i ditt tilfelle også kutte en del whitespace i html filene, men dette er mindre viktig og gjør bare siden bittelittegrann raskere, noe som ikke er noe tema på en såpass liten og grei side.
Flash saken klarer du fint å bytte ut med noe annet, så det kommer vel etterhvert!
Ellers er det bra
