@emestabillo
Posted
Hey Renzo, looks close to the design, congrats!
-
To get it closer still: the header image and the text inside the gallery section are missing the dark gradient that makes the text pop out. You'll see it if you look at the screenshot comparison.
-
Mobile toggle button should be a
buttoninstead of div -
I also noticed that you used a lot of px units. It's recommended to use scalable, relative units such as rem or em for accessibility
-
There are no hover effects on the footer, which I find odd since the rest of the interactive elements have it
-
There are items to address in the accessibility report
Hope this helps!
Marked as helpful
@ralvarezrenzo
Posted
@emestabillo Right now I will try to fix all the parts that you mentioned and I will start to investigate about the px em rem that I see that is very important, you are pleased to take your time you help me a lot to improve , thank you very much :D !
