Design comparison
Solution retrospective
My first one, please be kind!
Anselvi!
Community feedback
- @ecemgoPosted over 1 year ago
Some recommendations regarding your code that could be of interest to you.
CSS
- If you want to center it easily, you can update your
body
in this way:
body { background-color: hsl(212, 45%, 89%); font-family: "Outfit", sans-serif; display: flex; flex-direction: column; justify-content: center; align-items: center; min-height: 100vh; }
- if you use flexbox, you don't need to give margin to
whitebox
. Thiswhitebox
is the updated version:
.whitebox{ position: relative; height: 580px; max-width: 350px; // max-width makes it responsive background-color: hsl(0, 0%, 100%); border-radius: 5%; }
Hope I am helpful. :)
Marked as helpful0@AnselviPosted over 1 year agoThanks very much @ecemgo , just edited as you suggested and works great, also read on MDN about Flex and learned a new CSS property.
Thanks very much!!!
1@ecemgoPosted over 1 year ago@Anselvi Learning never ends in frontend development, we learn something new almost every day😉 I'm glad that I made this possible🤗
0 - If you want to center it easily, you can update your
- @neew18Posted over 1 year ago
Hello! You did well on ur first challenge.
https://www.octobot.io/blog/html-accessibility/ Hope this article above will help you with fixing those accessibility reports.
And for centering the white container, I think flex-boxes is best way. Please take a look at those.
I’m still learning too. Have a nice day.
Marked as helpful0@AnselviPosted over 1 year agoThanks very much @neew18 , I appreciate your kind words, I-ll take a look and keep on improving!
0
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