re: Portfolio Review VIEW POST

FULL DISCUSSION
 

In general, I really like it! Some things I personally don't:

  • All the page titles are Home |
  • There's not / Home link on the navbar
  • The background image is huge (use tinypng.com/)
  • It does not adjust well on different screen sizes well
  • Seems like it was made to look good on mobile but /about lacks here (the images shrink way too much
  • /about and /blog don't look hot on a wide screen (1080p even).
  • The nav link expansion was cool but got old fast (for me)
  • Add a link to dev.to/dylanesque :D

Hope this helps 😄

 

This is definitely helpful! I only noticed the Home issue tonight, that has been resolved with my latest commit. The lack of Home navigation was deliberate, but enough people have commented so that it needs to be addressed in my next commit. Duly noted with everything else you said, with one point of clarification: what do you mean by the nav expansion, the transition that occurs on hover for the nav links?

 

how when hovering over links in the navbar, the text gets bigger

So you think a simple color change to indicate th hover state is sufficient; got it, and thanks again!

code of conduct - report abuse