Landing a develop job: Portfolio

Hi,
it’s pretty cool but think to make disappear the scrollbar and correctly center the main card on the height.Capture d’écran (45)

Great feedback @MichKael, I think you’re right. Thank you!

1 Like

It looks great Nicholas!

I think maybe adding a transition: right #s property to the squares would make the hover animation of them coming out from behind the main card look a little smoother.

On mobile, from an accessibility and clarity standpoint, maybe there should be text next to the icons for each menu option?

You’re right, that was something I actually had in my CSS but had commented out during during testing… thanks for the reminder!

As for the mobile clarity, I had that feedback from a friend as well, so I decided to go with the crowd and make the menu items visible. Easy to forget other people haven’t stared at the icons for a bajillion hours and so don’t inherently know what they mean!

Thanks for the feedback!