Replex - grafikus
Verder over je code:
je hebt 2x de <html> tag in je pagina staan, ik heb nog niet op validiteit gecontroleerd, maar weet nu al dat dat niet goed zit. Naar kijken nog dus.
Layout zelf ziet er overigens mooi uit.
Skip knopje is idd wel handig :)! Vergeten ofzo.. Dat herladen van het menu is idd storend.. kweet ook niet zo goed hoe ik dat eruit ga krijgen. (hij past je adresbalk aan.. waardoor de pagina herlaad. etc.) En de dubbele <HTML> tag komt doordat ik de pagina's inlcude. Waardoor de CSS link niet werkt dus daarom heb ik gewoon de hele <HTML> zooi er weer ingegooit, iemand suggesties?
Alles wat je include moet werken met die css, hij geeft uiteindelijk maar 1 html pagina terug ;)
Menu items zijn een beetje moeilijk te lezen -> iets groter of meer ruimte tussen letters.
Haal die animatie uit de header is storend tijdens het lezen.
¿Paperclipje <> rijschool?
"Wij laten je niet zakken" mag wat groter
Streepje onder leerlingen mag vind ik weg
Helemaal onderop staat een zwartlijntje onder het grijze vlak en eentje onder het oranje lijntje, hoort dit?
Copyright <> Nederland. Zou het sowieso als tekst en niet als plaatje plaatsen.
Misschien is het leuker om een Nederlands weg te nemen, alhoewel daar staat altijd file ;-)
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
HTML Validation Result
----------------------
http://www.grafikus.nl/replex/index2.php
line 1 column 1 - Warning: missing <!DOCTYPE> declaration
line 20 column 9 - Warning: <embed> is not approved by W3C
line 20 column 175 - Warning: discarding unexpected </embed>
line 27 column 7 - Warning: discarding unexpected </div>
line 41 column 4 - Warning: discarding unexpected <!DOCTYPE>
line 42 column 1 - Warning: discarding unexpected <html>
line 43 column 1 - Warning: discarding unexpected <head>
line 40 column 7 - Warning: <title> isn't allowed in <td> elements
line 40 column 7 - Warning: <meta> isn't allowed in <td> elements
line 40 column 7 - Warning: <style> isn't allowed in <td> elements
line 57 column 1 - Warning: discarding unexpected <body>
line 8 column 1 - Warning: missing </div>
line 7 column 7 - Warning: missing </center>
line 11 column 3 - Warning: <table> proprietary attribute "height"
line 29 column 7 - Warning: <td> proprietary attribute "background"
line 40 column 7 - Warning: <td> proprietary attribute "background"
line 40 column 7 - Warning: <td> proprietary attribute "alt"
0 errors / 17 warnings
----------------------
http://www.grafikus.nl/replex/index2.php
line 1 column 1 - Warning: missing <!DOCTYPE> declaration
line 20 column 9 - Warning: <embed> is not approved by W3C
line 20 column 175 - Warning: discarding unexpected </embed>
line 27 column 7 - Warning: discarding unexpected </div>
line 41 column 4 - Warning: discarding unexpected <!DOCTYPE>
line 42 column 1 - Warning: discarding unexpected <html>
line 43 column 1 - Warning: discarding unexpected <head>
line 40 column 7 - Warning: <title> isn't allowed in <td> elements
line 40 column 7 - Warning: <meta> isn't allowed in <td> elements
line 40 column 7 - Warning: <style> isn't allowed in <td> elements
line 57 column 1 - Warning: discarding unexpected <body>
line 8 column 1 - Warning: missing </div>
line 7 column 7 - Warning: missing </center>
line 11 column 3 - Warning: <table> proprietary attribute "height"
line 29 column 7 - Warning: <td> proprietary attribute "background"
line 40 column 7 - Warning: <td> proprietary attribute "background"
line 40 column 7 - Warning: <td> proprietary attribute "alt"
0 errors / 17 warnings
niek schreef op 29.12.2006 18:22:
Hele mooie website, behalve dit dan:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
HTML Validation Result
----------------------
http://www.grafikus.nl/replex/index2.php
line 1 column 1 - Warning: missing <!DOCTYPE> declaration
line 20 column 9 - Warning: <embed> is not approved by W3C
line 20 column 175 - Warning: discarding unexpected </embed>
line 27 column 7 - Warning: discarding unexpected </div>
line 41 column 4 - Warning: discarding unexpected <!DOCTYPE>
line 42 column 1 - Warning: discarding unexpected <html>
line 43 column 1 - Warning: discarding unexpected <head>
line 40 column 7 - Warning: <title> isn't allowed in <td> elements
line 40 column 7 - Warning: <meta> isn't allowed in <td> elements
line 40 column 7 - Warning: <style> isn't allowed in <td> elements
line 57 column 1 - Warning: discarding unexpected <body>
line 8 column 1 - Warning: missing </div>
line 7 column 7 - Warning: missing </center>
line 11 column 3 - Warning: <table> proprietary attribute "height"
line 29 column 7 - Warning: <td> proprietary attribute "background"
line 40 column 7 - Warning: <td> proprietary attribute "background"
line 40 column 7 - Warning: <td> proprietary attribute "alt"
0 errors / 17 warnings
----------------------
http://www.grafikus.nl/replex/index2.php
line 1 column 1 - Warning: missing <!DOCTYPE> declaration
line 20 column 9 - Warning: <embed> is not approved by W3C
line 20 column 175 - Warning: discarding unexpected </embed>
line 27 column 7 - Warning: discarding unexpected </div>
line 41 column 4 - Warning: discarding unexpected <!DOCTYPE>
line 42 column 1 - Warning: discarding unexpected <html>
line 43 column 1 - Warning: discarding unexpected <head>
line 40 column 7 - Warning: <title> isn't allowed in <td> elements
line 40 column 7 - Warning: <meta> isn't allowed in <td> elements
line 40 column 7 - Warning: <style> isn't allowed in <td> elements
line 57 column 1 - Warning: discarding unexpected <body>
line 8 column 1 - Warning: missing </div>
line 7 column 7 - Warning: missing </center>
line 11 column 3 - Warning: <table> proprietary attribute "height"
line 29 column 7 - Warning: <td> proprietary attribute "background"
line 40 column 7 - Warning: <td> proprietary attribute "background"
line 40 column 7 - Warning: <td> proprietary attribute "alt"
0 errors / 17 warnings
Dit zijn slechts warnings en het is al aangehaald in het topic dat het geen valide HTML is.
Over de intro: de dingen die hier al gezegd zijn, dus o.a. inkorten en of verwijderen
Over het menu: Zeer mooi, maar die buttons links de tekst daarvan zou ik een iets andere kleur dan zwart geven, miss in de richting van wit?? De animatie (uitvouwen) zou ik zeker weglaten, erg irri.
Over de content deel: Niks op aan te merken erg mooie style.
Veel te traag, ik heb wel een idee hoe het eignelijk op een wat snellere moet zijn (mooi dus) maar dat is het nu niet.
En de intro zou ik misschien zelfs weghalen? Ben persoonlijk niet zo iemand die daar graag naar kijkt.
@Robert: Ik weet het, het was ook als " aanvulling " bedoelt.
2-> zielig voor de mensen die een trage computer hebben
Best wel mooie site:)
Maar mischien 1 of 2 kleurtjes meer en hij is nog wat beter:) Vind ik dan.
GR. mebus
Je title en de zin in de header verschillen.
wij laten je niet zakken ! en wij laten je niet zakken!
Ik zou voor de laatste gaan dus geen spatie voor !
En de intro ja..... dat is al wel gezegd
