Nieuwe portfolio
http://ninplay.nl/portfoliov2/
Als je jouw kleuren invult en de contrast waarde is niet groot genoeg dan is het verstanding om de kleuren aan te passen. Bijvoorbeeld door de lichtste van de twee lichter te maken en de donkerste donkerder.
- Menu veel te groot lettertype, staat er nu zo schreeuwerig
- groene kleur in combinatie met wit en grijs staat er goed! Alleen niet die 14 andere kleurtjes in de borders
- Linear gradient in de content staat niet mooi en hoort ook niet bij dit design
- tekst in header erg weinig contrast. Maakt dit beter: http://contrastrebellion.com/
- Bij menu hover komt er opeens een andere groene kleur bij, die ik niet zo vind passen bij de rest.
- zorg dat het logo klikbaar wordt en dat je dan naar de homepagina terug keert, dit is normaal
CODE
- Yahhoeeee! HTML5!!! :D
- Je moet altijd direct na <head> je charset definiëren: <meta charset=utf-8>
- .container1 en .container2, verzin beter namen voor je classes. Omschrijf zo goed mogelijk wat ze voorstellen.
- #featuredgradient onnodig, plaats dit gewoon als background-image van het #featured element
- gebruik geen <b> tags meer, maar <strong> tags.
- alle .kleinkleur en .grootkleur element overbodig. Gebruik border-image hiervoor (of anders psuedo elementen)
- Recente werk is een lijst, gebruik hiervoor dus <ul> en <li>
- probeer je te houden aan 1 H1 element per pagina, en zoveel mogelijk uniek. Stop dus alleen de belangrijkste titel in een H1 element, in dit geval is dat Welkom op mijn portfolio. De rest hoort in lagere headings.
- gebruik het <section> element voor het featured gedeelte en het <article> element voor het artikel gedeelte.
- gebruik <aside> voor de sidebar
- bij .grootkleur in je CSS: width: px; dit is fout
- gebruik shorthand codes voor margin, padding, border, enz.
- Haal je code even door http://prefixr.com/ en zorg dat je overal de juiste prefixes gebruikt
Toevoeging op 05/03/2012 16:53:02:
Dankje, Wouter. Zal morgen even uigebreid doornemen en dingen aanpassen. Wel wil ik even een opmerking maken over
Code (php)
1
width: px;
Dit zorgt ervoor dat het doorloopt over de hele breedte, hoe ver je ook uitzoomt. Het enige alternatief dat ik hierop zou kunnen verzinnen is width: 100%;, maar ik zou niet weten wat voor verschil dit maakt?
Nogmaals bedankt, morgen neem ik alles even beter door, stagedag afgelopen over 10 minuten.
Wat je nu hebt staan is invalid en fout. Omdat het fout is schiet de browser standaard in de default style voor dat element terug en dat is width: 100%; oplossing is dus of dit helemaal weghalen, of width: 100%; neerzetten, of width: auto; plaatsen.
Toevoeging op 06/03/2012 09:19:24:
Voor het logo zou ik trouwens graag willen dat je er met je muis overheen gaat, hij wat donkerder wordt. Ik bedoel geen hover met een andere kleur, maar dat hij langzaam wat donkerder wordt. Ik neem aan dat dit met Jquery gedaan moet worden?