Design comparison
Solution retrospective
suggestions welcomed...
Community feedback
- @adarshcodesPosted almost 4 years ago
Hi @mansi05041, Your solution is pretty good. You can work on the responsiveness part and if you add a transition for the button it would look nicer. Keep it up. Happy Coding.
2@mansi05041Posted almost 4 years ago@adarshcodes thanks for suggestion . i will work on it.
0 - @AryaAnish121Posted almost 4 years ago
Hi Mansi,
One suggestion:-
- you can give padding to the whole page
Happy Coding!
2 - @RenszCamachoPosted almost 4 years ago
Hi, mansi.
-illustration mockup SVG is broken, you miss-spelled the name, you miss the letter i. Check it out.
<img src="./images/illustration-mockups.svg" alt="mobile" class="mockups">
I think this is the issue with the background:
`@media screen {
- { color: white; background-image: url("images/bg-desktop.svg"); background-repeat: no-repeat; background-color: hsl(257, 40%, 49%); margin: 5px; padding: 5px; } }`
Actually, I do not understand that media query.
You can do like so:
body { height: 100vh; width: 100%; background-image: url("images/bg-desktop.svg"); background-repeat: no-repeat; background-color: hsl(257, 40%, 49%); background-size: cover; }
Hopefully, it helps you. 😊
Happy coding🧑💻
2@RenszCamachoPosted almost 4 years ago@mansi05041 I forgot to specify that you will remove the 'media-queries'. My bad.
2
Please log in to post a comment
Log in with GitHubJoin 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