Archief - PORTFOLIO: Unrated.be

Het archief is een bevroren moment uit een vorige versie van dit forum, met andere regels en andere bazen. Deze posts weerspiegelen op geen enkele manier onze huidige ideeën, waarden of wereldbeelden en zijn op sommige plaatsen gecensureerd wegens ontoelaatbaar. Veel zijn in een andere tijdsgeest gemaakt, al dan niet ironisch - zoals in het ironische subforum Off-Topic - en zouden op dit moment niet meer gepost (mogen) worden. Toch bieden we dit archief nog graag aan als informatiedatabank en naslagwerk. Lees er hier meer over of start een gesprek met anderen.
U

Unrated

Legacy Member
http://www.unrated.be/

Nieuwe site, een tijdje aan bezig geweest aan de coding!

Design is redelijk simpel maar dat was dan ook de bedoeling :)

Comments zijn welkom :):applause:
R

Rvl

Legacy Member
voor ge aan u coding begon had je mssh best es gezien hoe xhtml juist moet geimplementeerd worden ;). U menu klopt niet echt, je kan beter <ul><li> .. structuur gebruiken. Verder gebruik je ook onnodige divs. Valt me eigelijk op als ik zo eens snel naar u code kijk.
Qua layout is het niet slecht maar ik vind het nogal onhandig om te lezen. Al u onderbroken strepen, verschillende strepen, etc veroorzaken een drukke weergave en das toch nie wa je wil
:offtopic: wat me verder ook opvalt is dat je overvoedig gebruik maakt van smiley's, en vooral die bewegende zijn irritant
U

Unrated

Legacy Member
wat is er dan mis met die <ul>? Die mag je toch gebruiken in xhtml?
<ul>
<li>List1</li>
</ul>

Bedankt voor the reply :)
X

Xavez

Legacy Member
Code:

° xhtml: Te veel nutteloze divs. Dan had je het ding evengoed met tables kunnen opbouwen want op deze manier valt het nut van divs al volledig weg natuurlijk.
° xhtml: Je homepage is wel xhtml valid, maar om één ding te noemen: je contact-pagina alweer niet (en daarvoor moet ik niet eens de validator gebruiken). In je form staat ACTION in caps. xhtml heeft geen caps :)...
° xhtml/accessibility: Je menu is met afbeeldingen opgebouwd. Dat is ook hier volledig onnodig en zelfs weer een doorn in het oog van elke screenreader. Te meer om dat je de afbeeldingen al geen ALT-tags meegegeven hebt! Het laat ook niet toe om bijvoorbeeld de font-grootte in je browser in te stellen (slechtzienden)

° CSS: uw a: elementen zijn niet volgens de LoVeHAte volgorde gezet.
° CSS: iets als Courier New moet best tussen "" staan. (muggenziften, maar als je het strict wilt doen, doe het dan ook heel strict hé ;))

° PHP: uw implementatie lijkt me nogal raar. Waarom niet gewoon met invludes gedaan en met aparte pagina's gewerkt? URL's met ?= enzovoort zijn nogal internetonvriendelijk (zeker voor zo'n kleine site). Je kan beter een pagina index.php, portfolio.php etc etc aanmaken en dan met een header en footer werken.

° Ontwerp: ziet er wel strak en eenvoudig uit, alleen storen die ouderwets lelijke fades links en rechts mij mateloos. Zeker met die dubbele border er nog eens langs!


Dat is in een notendop wat ik zo op het eerste zicht zie. Fixen die handel! :)

edit

Unrated zei:
wat is er dan mis met die <ul>? Die mag je toch gebruiken in xhtml?
<ul>
<li>List1</li>
</ul>
Klik om te vergroten...
Is niks mis mee, maar Rvl bedoelt dat je voor je menu lijstjes moet gebruiken, in plaats van een hoop divs naast elkaar. Dat lijstje kan je dan perfect met css vormgeven gelijk je maar wil!
U

Unrated

Legacy Member
- Contact Page is valid nu! Code opgekuist

- Aan de menu's wordt gewerkt, all veel commentaar opgekregen :)

- PHP: Ik werk wel met includes maar doe dit via een php switch "$_GET['act']" en dan beslissen welke pagina te includen

- Bedoel je met die fades links en rechts de shadows van men borders?

- Ik heb de <img> in een <div> gezet om dat in IE er een space tussen de <img> genomen werd. Wrs wel op te lossen via css maar nog niet tegengekomen :)

Bedankt voor jullie comments

EDIT: Er zijn nog wat pagina's niet valid dit omdat mijn admin panel nog niet 100% werkt en dus nog niet de juiste xhtml in de database plant

EDIT2: Lovehate toegepast
X

Xavez

Legacy Member
Unrated zei:
- PHP: Ik werk wel met includes maar doe dit via een php switch "$_GET['act']" en dan beslissen welke pagina te includen

- Bedoel je met die fades links en rechts de shadows van men borders?

- Ik heb de <img> in een <div> gezet om dat in IE er een space tussen de <img> genomen werd. Wrs wel op te lossen via css maar nog niet tegengekomen :)
Klik om te vergroten...

1) Ik weet dat je met includes werkt, maar je doet het omgekeerd eigenlijk - vaak gebruikt door "beginners", dat wel. Je houdt één pagina index.php waar je dan al de rest inlaadt. Je kan beter een template maken header.inc en footer.inc en dan op elke pagina die je aanmaakt de header en footer includen. Zo hou je ook een meer logische opbouw van je website. Even goed qua mappenstructuur en dergelijke. die header en footer steek je dan bvb samen met al je images en stylesheets in je /template folder ofzo. Voordeel is dat elke pagina dan meer een aparte pagina lijkt, dus dat je contact.php, index.php, portfolio.php enzovoort krijgt, wat je url weer een pak meer interpreteerbaar maakt door zoekmachines e.d.

2) die fades inderdaad. Op zich is er niets mis met fades, hoor, maar met die extra border er nog eens rond heb ik het gevoel dat ik naar zo'n standaard-border van een tabel in internet-explorer zit te kijken!

3) ik snap waarom je die img in een div gezet hebt, maar je kan ze dan evengoed een margin van 0 meegeven. Geef ze ook allemaal hun ALT tag mee, zodat de navigatie ten minste leesbaar blijft voor mensen bij wie de afbeeldingen niet inladen, of die geen afbeeldingen kunnen zien (blinden en slechtzienden as said)

Leuk dat je actief jouw site verbetert, trouwens :).
R

Rvl

Legacy Member
u header
HTML: 
<div class="header">
		<img src="images/header.gif" alt="" border="0" />
	</div>
kunt ge gewoon vervangen door <div class="header"></div> en zet u achtergrond image gewoon in u css u heel menu van <div id="menu"> <div .... </div> </div> vervangen door <ul><li><a></a></li> ... </ul> structuur (eventueel met nog een wrapper div rond u <ul> </ul>
dit zijn zowat de "belangrijkste" dingen die ik zou aanpassen aan u opmaak. De browser zal geen fouten weergeven (zoals ge zelf kunt zien) maar bovenstaande structuren zijn hier beter voor

Is niks mis mee, maar Rvl bedoelt dat je voor je menu lijstjes moet gebruiken, in plaats van een hoop divs naast elkaar. Dat lijstje kan je dan perfect met css vormgeven gelijk je maar wil!
Klik om te vergroten...
ja dat bedoel ik dus

en zoals xavez zegt, mensen die hun sites actief verbeteren na rekening heben gehouden met comments, die helpen we graag verder (integendeel tot andere hier opt forum :gniffel: )
U

Unrated

Legacy Member
Bedankt voor alle comments al :) Kan alleen maar verbeteren! :P

Dus jullie zouden dit in een <div class="header"> zetten als background:
http://www.unrated.be/test.gif

En dan een <div class="menu"> in die header en met margins mooi afregelen?

In die menu div dan een <ul> met mijn links.

Zo dus:
HTML: 
<div class="header">
<div class="menu"><ul><li>Home</li><li>Portfolio</li></ul></div>
</div>
En met de css dan alles instellen zoals background en margins.

Mijn vraag is dan, hoe krijg ik die <ul> horizontaal? Want een list is toch bij default verticaal?
X

Xavez

Legacy Member
Je kan in plaats van classes ook beter met id's werken voor je divs. dat wordt dan voor je html:

HTML: 
<div id="header">
<h1>Unrated</h1>
<div id="menu"><ul><li>Home</li><li>Portfolio</li></ul></div>
</div>

en je css:

Code: 
#header {background: url('header.png') #<contrastrijke kleur met de kleur van je links>;}
#header h1 {visibility: hidden; font-size:0;}
#menu ul {display: inline;}

Even wat uitleg: die H1 is voor o.a. google en accessibility. Een h1-element geeft altijd de "hoogste" titel van je webpagina aan. Mensen die bvb geen header.png kunnen zien (css uitgeschakeld, blinde personen, zoekrobots,...), daarvoor wordt dan bvb wel via een screenreader of dergelijke het h1-element weergegeven, voorgelezen, opgenomen in de index etc...

Het komt er eigenlijk op neer dat je voor elk grafisch element een textueel equivalent moet hebben :).

en da's idd het soort structuur dat we zochten! :)
U

Unrated

Legacy Member
die <div id="menu"> is dan toch eigenlijk niet nodig?

Als ik gewoon <ul id="menu"> neem bespaar je toch ook wat code...

Anyway alvast bedankt voor alle hulp :)

Morgen is met bezig houden!

EDIT: Die #<contrastrijke kleur met de kleur van je links> wrm moet dit? Ik heb toch al achtergrond img? En de background color is in de body al geset
X

Xavez

Legacy Member
Unrated zei:
die <div id="menu"> is dan toch eigenlijk niet nodig?

Als ik gewoon <ul id="menu"> neem bespaar je toch ook wat code...

Anyway alvast bedankt voor alle hulp :)

Morgen is met bezig houden!

EDIT: Die #<contrastrijke kleur met de kleur van je links> wrm moet dit? Ik heb toch al achtergrond img? En de background color is in de body al geset
Klik om te vergroten...
Kan je inderdaad ook doen als je dat wil :). Bespaar je idd wat code mee - maar ik was te lui om m'n post te editen ^^.
Die contrastrijke kleur omdat het altijd wel eens kan voorvallen dat je achtergrondafbeelding niet inlaadt. Als dan je links in dezelfde kleur zijn als je header of dergelijke, wordt het heel moeilijk navigeren natuurlijk :). That's the only reason ;)
U

Unrated

Legacy Member
oke tx , morgen alles is proberen

ff nog een bericht aan de onbekende die me emailde over een veiligheids probleem! Blijkbaar konde ze mijn database details bemachtigen! Bedankt om hier op te wijzen zonder er misbruik van te maken! Probleem is opgelost ondertussen!
B

Breen

Legacy Member
Xavez zei:
1) Ik weet dat je met includes werkt, maar je doet het omgekeerd eigenlijk - vaak gebruikt door "beginners", dat wel. Je houdt één pagina index.php waar je dan al de rest inlaadt. Je kan beter een template maken header.inc en footer.inc en dan op elke pagina die je aanmaakt de header en footer includen. Zo hou je ook een meer logische opbouw van je website. Even goed qua mappenstructuur en dergelijke. die header en footer steek je dan bvb samen met al je images en stylesheets in je /template folder ofzo. Voordeel is dat elke pagina dan meer een aparte pagina lijkt, dus dat je contact.php, index.php, portfolio.php enzovoort krijgt, wat je url weer een pak meer interpreteerbaar maakt door zoekmachines e.d.
Klik om te vergroten...

Het gebruik van een frontcontroller al dan niet in samenwerking met (x)HTML templates voor volledige code scheiding is zeker niet enkel iets voor "beginners" Xavez, althans in PHP5 niet.

Misschien kan je (Unrated) eens htaccess & rewriting bekijken. Daarmee kan je je URL's leesbaar & aantrekkelijk maken voor zoekbots & je frontcontroller systeem blijven gebruiken.
U

Unrated

Legacy Member
oke hier ben ik weer :)

FF geprobeerd wat jullie voorstelden maar werkt niet echt zoals het moet :)

http://test2.unrated.be

De list word verticaal vertoond en niet horizontaal en ook niet op de plaats die ik instelde met de style :\

Enige hulp is welkom!
D

Dash

Legacy Member
Je Comments zijn wel raar gerangschikt eh, normaal gezien gaat:

Code: 
oudste comment
        .
        .
        .
nieuwste comment

bij jou is het:

Code: 
nieuwste comment
        .
        .
        .
oudste comment
R

Ramm

Legacy Member
Unrated zei:
oke hier ben ik weer :)

FF geprobeerd wat jullie voorstelden maar werkt niet echt zoals het moet :)

http://test2.unrated.be

De list word verticaal vertoond en niet horizontaal en ook niet op de plaats die ik instelde met de style :\

Enige hulp is welkom!
Klik om te vergroten...
Uw "li" moet float:left meekrijgen
X

Xavez

Legacy Member
Uw CSS-opbouw is mij wel een raadsel!
je geeft bijvoorbeeld je h1 een id mee en dan doe je #h1-id hi {}

CSS-opbouw werkt via sub-elementen. Je kan dus beter bijvoorbeeld #menu h1 {} doen. Op deze manier wordt de stijl tussen de {}-haken op alle h1-elementen in de div-layer met id "menu" toegepast. Snap je het verschil met wat jij doet?
m

menlor

Legacy Member
Vul uw alt van uw images in. Zo kunnen mensen die niet goed kunnen lezen toch nog wete waar de knop voor staat, aangezien alt wel uitvergrootbaar is en uw images ni.

thx
Het archief is een bevroren moment uit een vorige versie van dit forum, met andere regels en andere bazen. Deze posts weerspiegelen op geen enkele manier onze huidige ideeën, waarden of wereldbeelden en zijn op sommige plaatsen gecensureerd wegens ontoelaatbaar. Veel zijn in een andere tijdsgeest gemaakt, al dan niet ironisch - zoals in het ironische subforum Off-Topic - en zouden op dit moment niet meer gepost (mogen) worden. Toch bieden we dit archief nog graag aan als informatiedatabank en naslagwerk. Lees er hier meer over of start een gesprek met anderen.
Terug
Bovenaan