DEV Community

Discussion on: My Portfolio Site

Collapse
 
matijanovosel profile image
Matija Novosel • Edited

It is a nice site and I'd hate to nitpick but I'd suggest looking into a few things:

  • The link icons on the navigation stay as text when I hover over them and leave quickly (don't know if that's intentional)
  • The title of the page and routes all have the default React one, I'd recommend looking into dynamic changes of the page metadata and changing the default favicon as well
  • The blogs and timeline routes don't share the navigation and overflow both in width and height
  • The dev.to link points to one of your articles, don't know if you meant to point it at your profile
Collapse
 
jay818 profile image
Jayant

Thnx for the feedback bro 🙂
I will do the required chages

Collapse
 
georgewl profile image
George WL

I can recommend creating a simple hook that sets the document.title on page change 😁