Design comparison
Solution retrospective
All feedback is welcome, have a nice day
Community feedback
- @Zy8712Posted over 1 year ago
Looks pretty good. Some things I suggest you add if you want your code to appear slightly more similar to the original design:
body { min-height:100vh; /* this makes your box vertically align to the center */ background-color: hsl(212, 45%, 89%); }
.modificador h1{ text-align: center; }
.modificador p { text-align center; }
One other thing is that I would suggest you to use <h2> instead of <h1>. Besides that it looks like you a firm grasp on the fundamentals π.
0 - @0xabdulkhaliqPosted over 1 year ago
Hello there π. Congratulations on successfully completing the challenge! π
- I have other recommendations regarding your code that I believe will be of great interest to you.
CSS π¨:
- Looks like the component has not been centered properly. So let me explain, How you can easily center the component without using
margin
orpadding
.
- We don't need to use
margin
andpadding
to center the component both horizontally & vertically. Because usingmargin
orpadding
will not dynamical centers our component at all states
- You already using
Flexbox
for layout, but you didn't utilized it's full potential. Just add the following rules to properly center the component.
body { min-height: 100vh; }
- Now remove these styles, after removing you can able to see the changes
body { margin: 1.25rem; margin-left: auto; }
- Now your component has been properly centered
.
I hope you find this helpful π Above all, the solution you submitted is great !
Happy coding!
0 - @roodhousePosted over 1 year ago
Hey. Great job! You have the spacing and padding/margins down really well. IMO you should utilize more container divs to help with the styling. For instance if you were to drop a <div> <div> <h1> then you would be able to manipulate the 2 higher up divs before you touched the h1. This would allow you to then see what styling is left to implement on h1. You might find one or two things or nothing is needed at all. Personally, I attempt to place the bulk of my styling on divs and if needed, then I style the data directly. Anyway, great start and keep it going!
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