Hejsa.
Det ser godt ud!
Her er dog lige et par ting som fangede mit øje:
1. Der er ikke noget Parallex over det design.
Godtnok skriver du "Parallex-agtigt" men det er der altså ikke.
(https://en.wikipedia.org/wiki/Parallax)
2. Du skriver de fleste af dine tekster i div
-tags, men de burde være i p
-tags.
Eks. <div class="velkomst">...</div>
3. "Keep It Simple Stupid" har du inline styling — overvej at flytte det ud i CSS filen og evt. pak det ind således:
<p><em>.kiss</em> { content: '<em>K</em>eep <em>I</em>t <em>S</em>imple, <em>S</em>tupid!' ;}</p>
4. Dine kompetencer har du lavet som en tabel, hvilket er nemt og virker efter hensigten.
En mere semantisk korekt måde ville være at skrive det i en dl
-liste:
<dl>
<dt class="photoshop">Photoshop</dt>
<dd>
<span class="skills skills-75">75%</span>
</dd>
...
</dl>
5. Portfoliobillederne er meget store og er resized i browseren — du vil kunne spare en del båndbredde ved at lave dem den rette størrelse.
6. Der er ingen tekst der beskriver dine sociale ikoner/links (Twitter, Github osv).
Skriv en tekst og skjul den med CSS (text-indent: -999px; overflow: hidden;)
7. Der er ikke nogle repositories på din Github profil?
8. Social ikonerne kunne laves som image-sprites, så slipper du for at de blinker når man holder musen over linket. Læg evt. dem alle sammen i den samme sprite, så skal du kun kalde ét billede frem for 8
9. Thumbs up for brug af CSS animation.