Tanya84 / portfolio

The starter portfolio repository, basically empty, but with a few helper files.
https://learn-the-web.algonquindesign.ca/courses/web-dev-5/
0 stars 0 forks source link

Peer Critique #7

Open AlexSpagnuolo opened 5 years ago

AlexSpagnuolo commented 5 years ago
0 points 1 points 2 points 3 points
Pages & images Barely started Missing lots of pages & images Has most of the pages, some placeholder images All pages exist and are well done with complete imagery
Personality & design Bland, boring, just another graphic designer Fairly generic, visual design could be pushed more Quality layouts and unique, personable content & design Unique and recognizable as you, with engaging, personable content & design
Image quality Only placeholder images Some portfolio piece images Lots of images but low quality High quality portfolio images — and lots of them
Accessibility No considerations Added some alt attributes, nothing else Just the basics: alt attributes, roles Accessibility is well considered and tested
Text content Fake content only Some real content, some fake All real content but with some of errors Real content, well written, no grammar or spelling errors
Responsiveness Not responsive Works on some screens Responsive but with some of awkwardness Looks great on all screen sizes
Code quality Barely started Indentation is barely existent, lots of validation errors, very poor semantics Decent indentation, just a couple validation errors, decent semantics Well indented, fully valid, good semantics

I love your illustrations featured on the website. More semantics and better indentation will benefit you. I also find the hover states of the portfolio pieces a bit odd, especially how the text changes colour. Try some more consistent animations and transition lengths throughout the site.