DEV Community

Discussion on: Portfolio Review

Collapse
 
nektro profile image
Meghan (she/her)

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 😄

Collapse
 
dylanesque profile image
Michael Caveney

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?

Collapse
 
nektro profile image
Meghan (she/her)

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

Thread Thread
 
dylanesque profile image
Michael Caveney

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

Thread Thread
 
nektro profile image
Meghan (she/her)

anytime, glad to help out 😄