search

derickdinh613 / derickdinhdesign

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

Critique #2

Open AlexSpagnuolo opened 6 years ago

AlexSpagnuolo commented 6 years ago

| 2 | 3 | 1 | 2 | 3 | | Creativeness | I have a glimpse at your brand but I feel like you can take it further through your site design | | Effectiveness | It effectively shows your work and is easily navigable | | Quality | Finish your footer, I see bullet points on the top right in your UL for the portfolio pieces on the home page. I do like your nav on the case study page though. I like how the images break the grid. | | Effort | You can take it further once all your written content and images are in. | | Code quality | Everything is indented, classes are used and the code looks clean. |

derickdinh613 commented 6 years ago

Thanks for the critique, I will take into consideration everything you've said!