@TekaNik
Posted
yeah, besides displaying both hero background img, you did good.
@devhalimah
Posted
Thanks a lot @TekaNik 😊
@davidomarf
Posted
Hey! The desktop layout it's pretty good. But in resolutions below 1284px, the image starts to overflow and becomes less and less visible. Then, around 1040, the image gets duplicated and fills all the width of the screen, looking streched.
You're displaying both the hero-desktop and hero-mobile images.
The only other detail, it's the button. It looks a liiiiitle bit bigger than the text field. In the prototype they're the same size.
Try to play with the property box-sizing: border-box
, and try to manually set the height of both the input field, and the button.
@devhalimah
Posted
Thanks @davidomarf I just fixed this. Please check again and let me know if there's anything I missed
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord