@Miculino
Posted
Congrats on completing the challenge, @PitagoraStrike!
I had a look at your final solution and Github repo. Here are a few suggestions that might help:
-
On larger screen sizes, there's a bit too much space at the top
-
Responsiveness needs a bit of polishing - you have horizontal scrolling on smaller screen sizes
-
Don't use fixed width and height, especially not with the px unit
-
You've used a lot of absolute positioning, which is really not needed in this case. You can build simple, modern, and efficient layouts and position your elements properly using Flexbox or CSS grid
Marked as helpful