Gå til innhold

Er dette et gunstig kodeoppsett? xhtml/css/php-include


Anbefalte innlegg

Hei. Driver å lærer meg php og diverse sammensetninger mellom det og xhtml&css for tida. Og da lurer jeg på om måten jeg har satt opp koden er bra/riktig, evnt om det bør settes opp på en annen måte.

 

index.php

 

<html>
<head><link rel="stylesheet" type="text/css" href="style.css" /></head>
<body>
<div id="banner"><?php include("banner.php"); ?></div>
<div id="menu"><?php include("menu.php"); ?></div>
<div id="content"><?php include("content.php"); ?></div>
<div id="footer"><?php include("footer.php"); ?></div>
</body>
</html>

 

style.css

 

a {
text-decoration: none;
color: black;
}

a:hover {
text-decoration: none;
color: #333333;
}
body { 
background-color: #444444;
}

#banner {
padding-top: 15px;
width: 800px;
height: 100px;
border: 1px;
border-style: solid;
border-color: black;
background-color: #4566A3;
font-size: 40px;;
text-align: center;
}

#menu {
width: 800px;
border: 1px;
border-top: 0px;
border-style: solid;
border-color: black;
background-color: #4566A3;
text-align: center;
padding: 4px;
padding-bottom: 0px;
}

#content {
width: 800px;
border: 1px;
border-top: 0px;
border-style: solid;
border-color: black;
background-color: #4566A3;
padding: 4px;
}

#footer {
width: 800px;
height: 100px;
border: 1px;
border-top: 0px;
border-style: solid;
border-color: black;
background-color: #4566A3;
padding: 4px;
}

 

menu.php

 

<a href="index.php">Home</a>  |  
<a href="info.php">Info</a>  |  
<a href="support.php">Support</a>   |  
<a href="contact.php">Contact</a>  |  
<a href="links.php">Links</a>

 

 

 

Testsiden ligger her

Vet den er ille, men det her er bare testing av koder.

Synes også css-koden blir voldsom lang. Noe jeg har gått glipp av i forhold til mulig forkorting av den?

 

 

Btw. når siden er i ie blir den 800px bred som angitt, mens i opera blir deler av den 808px bred (width 800px + padding 4px*2)

Endret av PepsiCo
Lenke til kommentar
Videoannonse
Annonse
Voldsomt lang CSS? Det der kan umulig være mer enn 50 linjer.

Er ikke så lang nå, men den føles sånn, er jo ingen classes og sånt på sida og om det kommer så blir det jo endel. Det er kanskje normalt?

 

Hva er vitsen med å bruke "PHP include" når du har en meny som leder til statiske PHP-filer?

For å teste ut, samt å kunne redigere f.eks menyen fra en fil om jeg vil forandre den. Noen mer gunstige metoder når det er statisk?

 

Sett opp menyen i en ul med li for hvert punkt.

Will do

Lenke til kommentar

Forkortet over 40 linjer CSS til nesten halvparten slik:

 

#banner, #menu, #content, #footer {

width: 800px;

border: 1px solid #000 ;

border-top: 0;

background-color: #4566A3;

}

 

#banner {

padding-top: 15px;

height: 100px;

font-size: 40px;

text-align: center;

}

 

#menu {

text-align: center;

padding: 4px 4px 0;

}

 

#content {

padding: 4px;

}

 

#footer {

height: 100 px;

padding: 4px;

}

 

Håper jeg i farten ikke utelot noe, dobbelsjekk.

Det er tidlig på morgenen her borte. (USA)

Endret av selurdog
Lenke til kommentar
Forkortet over 40 linjer CSS til nesten halvparten slik:

 

 

#banner, #menu, #content, #footer {

width: 800px;

border: 1px solid #000 ;

border-top: 0;

background-color: #4566A3;

}

 

#banner {

padding-top: 15px;

height: 100px;

font-size: 40px;

text-align: center;

}

 

#menu {

text-align: center;

padding: 4px 4px 0;

}

 

#content {

padding: 4px;

}

 

#footer {

height: 100 px;

padding: 4px;

}

 

 

Håper jeg i farten ikke utelot noe, dobbelsjekk.

Det er tidlig på morgenen her borte. (USA)

Ahh, så man kan sette det opp sånn ja, ser lovende ut :)

 

 

Husk at noe med id "banner" kan bli automatisk blokket av diverse Adblockers, da er det bedre å bruke et annet navn på id-en til elementet.
Høres fornuftig ut. Får redigere litt :)
Lenke til kommentar
Ser forresten ut som at du har en bug på sida:

post-111209-1199454558.jpg

 

Edit:

Det er forresten i Opera ;)

Ja, jeg så det. Skrev det i førstepost også

"når siden er i ie blir den 800px bred som angitt, mens i opera blir deler av den 808px bred (width 800px + padding 4px*2)"

 

Er det andre muligheter enn å fjerne padding da?

Endret av PepsiCo
Lenke til kommentar

Ahh, så ikke det. Trodde det var meningen at den skulle være sånn hehe

Det har antakeligvis noe med padding-greiene dine å gjøre...

 

Prøv dette:

* {

padding: 0px;

margin: 0px;

}

 

Men nei, jeg har ingen aning om det funker. Bare sett andre prøve det når lignende har skjedd..?

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å
  • Hvem er aktive   0 medlemmer

    • Ingen innloggede medlemmer aktive
×
×
  • Opprett ny...