search

digitalsara / 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 Review #4

Closed livbarton21 closed 5 years ago

livbarton21 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 lots of errors Real content, well written, no grammar or spelling errors
Responsiveness Not responsive Works on some screens Responsive but with lots 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

Sara's website looks awesome, I love the feature that makes the portfolio images change from black and white to colour as the hover effect. Great job!

21/21