Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

practices using just html and basic css

@elwolffy


Design comparison


SolutionDesign

Solution retrospective


this is my solution, feel free to give me your comments, advices, constructive criticism

Community feedback

@C-likethis123

Posted

There was a good attempt to make the design responsive for different screen sizes. Some issues I have noticed:

  • On smaller screen sizes, the text is supposed to align to the center, but the button continues to be aligned to the left even after there was a CSS rule to make the text align to the center. Instead of: h1, p, a { text-align: center; }

Try: #content { text-align: center; }

  • From screen sizes 390px to 950px, the background image doesn't exactly align with the edges of the screen. Try adding background-size: contain to fix that.
0

@elwolffy

Posted

thank you for the feedback i'll check it out

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

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