@gsterczewski
Posted
Hello Danny 👋, my name is Grzegorz.
Your work is ok, there is some work to do, but it's a good starting point.
Here's my suggestions, I hope they will be useful to you
- On desktop there is a not enough space for Stand out... heading, increasing width in this rule should help :
section .container { width: 55% }
-
I would add some
hover
andfocus
states on anchors innav
to improve a11y -
I would also add
focus
state on.brand-link
so hovering and navigating with keyboard would have the same effect. -
Consider using inline
svg
instead ofimg
in social media links, so you can apply betterhover
andfocus
effects with adjustingfill
property.
This are the things you can fix pretty fast, later we can focus on improving layout and general responsiveness.
Keep up the good work! 💪
See you on the coding trail 😉
Cheers!