Design comparison
Community feedback
- @HassiaiPosted almost 2 years ago
Replace <div class="container"> with a main tag in the html file. eg: <main><img><div class="text"><h1></h1><p></p></div></main> To center a content on a page add min-height:100vh; the body and remove height:100vh. Give the main a background-color of white, a width value , padding value and a border-radius. Give .text a padding top and bottom value and dont change the color of the paragragh reduce the opacity. eg: .text{ padding: 125em 0;} 1.25em= 20px.text p{ opacity: 0.7} Use rem or em as unit measure for ur padding , margin and preferably rem for the font-size for more on this watch this https://youtu.be/N5wpD9Ov_To in your media query give the main a width of 80%; and remove the height value. Hope am helpful HAPPY CODING
Marked as helpful1@mohamed-montaser1Posted almost 2 years ago@Hassiai yep i know but if i replace <div class="container"><div> with main tag in html i know it semaintic but frontendmentor will tell me "the app must have <div class="container"><div> to contain the whole website
thank you for your commit
HAPPY CODING
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