Advies herbouw site met css.

  • Onderwerp starter Onderwerp starter jebl
  • Startdatum Startdatum
Status
Niet open voor verdere reacties.

jebl

Gebruiker
Lid geworden
8 dec 2011
Berichten
109
Hallo,

Graag zou ik advies willen over het volgende:
Onlangs ben ik begonnen met het herbouwen van deze site, http://www.ghsv-gideon.nl/index.html.
Omdat het opgebouwd is met tabellen en het een zootje is in de html, ben ik begonnen met css.
Omdat ik een beginner ben met css vraag ik mij af of het een beetje klopt wat ik heb gemaakt tot nu toe.
Een test versie van de site is te vinden op: http://www.ghsv-gideon.nl/test/index.html.
Ook heb ik nog wat vragen wat betreft het positioneren, indelen en invullen van div's.
Graag jullie mening en advies.
Hieronder de betreffende code.

Code:
<!DOCTYPE html>

<head>
<meta charset=utf-8" />
<title>GHSV Gideon Groningen</title>
<link href="css/gideon-style.css" rel="stylesheet" type="text/css">
</head>
<body>

<div class="banner-left"></div>
<div class="banner-right"></div>
<div class="logo-gideon"></div>

<div class="navi">
<ul>
<li><a href="/home/" title="Home">Home</a></li>
<li><a href="/bestuur/" title="Bestuur">Bestuur</a></li>
<li><a href="/leden/" title="Leden">Leden</a></li>
<li><a href="/agenda/" title="Agenda">Agenda</a></li>
<li><a href="/uitslagen/" title="Uitslagen">Uitslagen</a></li>
<li><a href="/contact/" title="Contact">Contact</a></li>
<li><a href="/gastenboek/" title="Gastenboek">Gastenboek</a></li>
<li><a href="/links/" title="Links">Links</a></li>
<li><a href="/nieuws/" title="Nieuws">Nieuws</a></li>
<li><a href="/fotogalerij/" title="Fotogalerij">Fotogalerij</a></li>
<li><a href="/archief/" title="Archief">Archief</a></li>
</ul>    
</div>

<div class="content">
<!-- begin bewerkbaar gebied -->

  <p>Hier komt de inhoud van de site.</p>
  
<!-- einde bewerkbaar gebied -->
</div>

<div class="footer">
<div class="footer-left"><script type="text/javascript" src="js/onestat.js"></script></div>
<div class="footer-center"><script type="text/javascript" src="js/last-update.js"></script></div>
<div class="footer-right"><script type="text/javascript" src="js/copyright.js"></script></div>
</div>

</body>
</html>


Code:
body {
	background-attachment: fixed;
	background-image: url(../images/layout/backgr.jpg);
	background-repeat: no-repeat;
	background-position: center center;
	-webkit-background-size: cover;
	-moz-background-size: cover;
	-o-background-size: cover;
	background-size: cover;
	margin: 0 0 0 0;
}
.logo-gideon {
	background-image: url(../images/layout/banner-gideon.gif);
	height: 120px;
	width: 1000px;
	margin-left: auto;
	margin-right: auto;
	z-index: 1;
	position: relative;
}
.banner-left {
	background-image: url(../images/layout/banner-links.gif);
	height: 120px;
	width: 50%;
	float: left;
}
.banner-right {
	background-image: url(../images/layout/banner-rechts.gif);
	height: 120px;
	width: 50%;
	float: right;
}
.navi{
	width: 1000px;
	height: 16px;
	margin-left: auto;
	margin-right: auto;
	font-family: Arial, Helvetica, sans-serif;
	border-bottom: 1px solid white;
}
.navi ul {
	list-style-type: none;
	padding-left: 0px;
	padding-right: 0px;
	margin: 0;
	word-spacing: 41px;
}
.navi ul li {
	display: inline;
	margin: 0 0px;
}
.navi ul li a {
	color: #000;
	text-decoration: none;
	font-size: 14px;
}
.navi ul li a:hover {
	color:#FFF;
}
.content {
	font-family: Arial, Helvetica, sans-serif;
	font-size: 14px;
	height: 655px;
	width: 1000px;
	margin-left: auto;
	margin-right: auto;
	margin-top: 4px;
	overflow-x: auto;
}
.footer {
	width: 1000px;
	height: 20px;
	margin-left: auto;
	margin-right: auto;
}
.footer-left {
	height: 20px;
	width: 200px;
	font-size: 12px;
	text-align: left;
	float: left;
}
.footer-center {
	height: 20px;
	width: 600px;
	font-size: 12px;
	text-align: center;
	float: left;
	margin-left: auto;
	margin-right: auto;
}
.footer-right {
	height: 20px;
	width: 200px;
	font-size: 12px;
	text-align: right;
	float: left;
}
 
Ik heb nu een jaar ervaring in CSS en HTML, de structuur ziet er goed uit.

Een paar opmerkingen:

Om alles goed te plaatsen kun je het beste alle content in een container zetten zoals:

[CPP]<html>
<head>
<body>
<div class="container">
<div class="banner-left"></div>
<div class="banner-right"></div>
<div class="logo-gideon"></div>
<div class="navi"></div>
<div class="content"></div>
<div class="footer"></div>
</div>
</body>
</html>[/CPP]

Voor de stevigheid is het handig om banner-left, banner-right en logo-gideon in een 1 div te zetten zodat deze 3 divs nooit zomaar weg kunnen springen maar altijd vast zitten in de hoofd div zoals in dit voorbeeld:

[CPP]<html>
<head>
<body>
<div class="container">
<div class="header">
<div class="banner-left"></div>
<div class="banner-right"></div>
<div class="logo-gideon"></div>
</div>
<div class="navi">
<div class="content">
<div class="footer">
</div>
</body>
</html>
[/CPP]

Wat de CSS betreft een paar punten:

Ik zie dat je bij de body geen width hebt gedefinieerd, wat ik wel aanraad is dat je de body een width van 100% geeft. Ik denk namelijk dat je die banner over het hele scherm wilt hebben, en bij grotere beeldschermen met meer dan 1000px wordt de banner niet over het hele scherm gevuld.

bij navi zie ik dat de border over de menu items komen dit kun je makkelijk oplossen zoals hier:

[CPP]
.navi {
border-bottom: 1px solid #FFFFFF;
font-family: Arial,Helvetica,sans-serif;
height: auto; /*Zodat de height van de div altijd goed staat*/
margin-left: auto;
margin-right: auto; /*margin-left en margin-right op auto zodat de div van het menu in het midden staat*/
width: 1001px;
}[/CPP]

Voorderest ziet het netjes uit.
 
Laatst bewerkt:
Hallo codGmer,

Bedankt voor het checken.
Zoals je ziet staan de drie div's van de "footer" in een container, dit wou niet lukken met de banner.
Wellicht ligt dit aan mijn kennis van css maar ik kreeg de drie afbeelding niet op dezelfde hoogte in de banner.

"Navi" en "body" heb ik aangepast, ik begrijp alleen niet waarom een width van 1001px voor navi.
Het verdelen van het menu over de totale breedte (1000px) van de div lukte niet, dus dat heb ik opgelost met word-spacing 41px.

Ook lukt het me niet om de welkomst tekst netjes in de div van het logo te krijgen, de tekst komt er buiten te staan.
Als ik hier een aparte div voor maak krijg ik hem niet op de goede positie.
 
Laatst bewerkt:
Inmiddels een nieuwe versie online gezet met de volgende code's:

HTML:
[CPP]<!DOCTYPE html>

<head>
<meta charset=utf-8" />
<title>GHSV Gideon Groningen</title>
<link href="css/gideon-style.css" rel="stylesheet" type="text/css">
</head>
<body>

<div class="header">

<div class="banner-left"></div>
<div class="banner-right"></div>
<div class="logo-gideon"><h1>Welkom op de website van Gebruikshondensportvereniging Gideon Groningen</h1></div>
</div>



<div class="navi">
<ul>
<li><a href="/home/" title="Home">Home</a></li>
<li><a href="/bestuur/" title="Bestuur">Bestuur</a></li>
<li><a href="/leden/" title="Leden">Leden</a></li>
<li><a href="/agenda/" title="Agenda">Agenda</a></li>
<li><a href="/uitslagen/" title="Uitslagen">Uitslagen</a></li>
<li><a href="/contact/" title="Contact">Contact</a></li>
<li><a href="/gastenboek/" title="Gastenboek">Gastenboek</a></li>
<li><a href="/links/" title="Links">Links</a></li>
<li><a href="/nieuws/" title="Nieuws">Nieuws</a></li>
<li><a href="/fotogalerij/" title="Fotogalerij">Fotogalerij</a></li>
<li><a href="/archief/" title="Archief">Archief</a></li>
</ul>
</div>

<div class="content">
<!-- begin bewerkbaar gebied -->

<p>Hier komt de inhoud van de site.</p>

<!-- einde bewerkbaar gebied -->
</div>

<div class="footer">
<div class="footer-left"><script type="text/javascript" src="js/onestat.js"></script></div>
<div class="footer-center"><script type="text/javascript" src="js/last-update.js"></script></div>
<div class="footer-right"><script type="text/javascript" src="js/copyright.js"></script></div>
</div>

</body>
</html>
[/CPP]

CSS:
[CPP]body {
background-attachment: fixed;
background-image: url(../images/layout/backgr.jpg);
background-repeat: no-repeat;
background-position: center center;
-webkit-background-size: cover;
-moz-background-size: cover;
-o-background-size: cover;
background-size: cover;
margin: 0 0 0 0;
width: 100%;
height: 100%;
}
h1 {
font-family: Arial, Helvetica, sans-serif;
color: #FFF;
font-size: 16px;
text-align: center;
position: relative;
top: -72px;
font-style: oblique;
font-weight: normal;
margin-top: 0px;
}


.logo-gideon {
background-image: url(../images/layout/banner-gideon.gif);
height: 120px;
width: 1000px;
margin-left: auto;
margin-right: auto;
z-index: 1;
position: relative;
font-size: 16px;
text-align: right;
color: #FFF;
font-family: Arial, Helvetica, sans-serif;
}

.banner-left {
background-image: url(../images/layout/banner-links.gif);
height: 120px;
width: 50%;
float: left;
z-index: 0;
position: relative;
}
.banner-right {
background-image: url(../images/layout/banner-rechts.gif);
height: 120px;
width: 50%;
float: right;
z-index: 0;
position: relative;
}
.navi{
width: 1001px;
height: auto;
margin-left: auto;
margin-right: auto;
font-family: Arial, Helvetica, sans-serif;
border-bottom: 1px solid white;
}
.navi ul {
list-style-type: none;
padding-left: 0px;
padding-right: 0px;
margin: 0;
word-spacing: 41px;
}
.navi ul li {
display: inline;
margin: 0 0px;
}
.navi ul li a {
color: #000;
text-decoration: none;
font-size: 14px;
}
.navi ul li a:hover {
color:#FFF;
}
.content {
font-family: Arial, Helvetica, sans-serif;
font-size: 14px;
height: 655px;
width: 1000px;
margin-left: auto;
margin-right: auto;
margin-top: 4px;
overflow-y: auto;
}
.footer {
width: 1000px;
height: 20px;
margin-left: auto;
margin-right: auto;
}
.footer-left {
height: 20px;
width: 200px;
font-size: 12px;
text-align: left;
float: left;
}
.footer-center {
height: 20px;
width: 600px;
font-size: 12px;
text-align: center;
float: left;
margin-left: auto;
margin-right: auto;
}
.footer-right {
height: 20px;
width: 200px;
font-size: 12px;
text-align: right;
float: left;
}
[/CPP]
 
Met de welkomst tekst is het nu ook gelukt, ik ben benieuwd of ik het goed heb gedaan.
 
Zou er iemand zo vriendelijk willen zijn deze site te checken.
Ik ben benieuwd :) en alvast bedankt.
 
Je bent <body> vergeten...
Maak altijd eerst de html/css voor de structuur en laat de tekst eruit.
Als de structuur in orde is ga je blok voor blok invulling geven.
Zet elke <div> en </div> op een eigen regel zodat je de structuur kunt teruglezen.
Geef commentaar bij de </div> van een 'groot blok', bijvoorbeeld
<div id="content-tekst">
.....
</div> <!--/content-tekst-->

Ga naar W3Schools voor de css :d
 
Hoi bron,

Bedankt voor het checken :thumb:
Die body tag is er waarschijnlijk uit gevallen met het aanpassen en wijzigen :o
Wat bedoel je met,
Maak altijd eerst de html/css voor de structuur en laat de tekst eruit.
Als de structuur in orde is ga je blok voor blok invulling geven.
?
Ik dacht dat de structuur wel in orde was.
Zo niet zou ik graag wat meer uitleg krijgen :)
 
Hier zijn boeken vol over geschreven. Uitleg hierover is niet in een berichtje samen te vatten. Gewoon googelen en veel lezen.

30 HTML Best Practices
http://code.tutsplus.com/tutorials/30-html-best-practices-for-beginners--net-4957

30 CSS Best Practices
http://code.tutsplus.com/tutorials/30-css-best-practices-for-beginners--net-6741

Writing Your Best Code
http://learn.shayhowe.com/html-css/writing-your-best-code/

Page Layouts
http://www.iraqtimeline.com/maxdesign/basicdesign/principles/prinlayout.html

En nog veel meer websites......
 
Oké, ik zal me er eens in verdiepen en zien of ik er wat van op kan steken.
In ieder geval bedankt voor je feedback :thumb:
 
Status
Niet open voor verdere reacties.
Terug
Bovenaan Onderaan