Nikola D• 1,450
@NikolaD93
Posted
Hello there! 👋
Congrats for completing the challenge! Everything looks great and the code is clean. 🙌
Hope this was helpful for you!
Happy holidays! 🎉✌
Marked as helpful
1
Design comparison
Community feedback
Hassia Issah• 50,670
@Hassiai
Posted
Replace<div class="fullcontainer">with the main tag,<h2> with <h1> and <div class="attribution"> with the footer tag to fix the accessibility issues. for more on web-accessibility [**click here **] (https://uxdesign.cc/semantic-html-the-foundation-of-web-accessibility-e5bbecad7c17)
Give .textDiv a padding value for all the sides, give p margin-top value or h1 margin-bottom value for the space between the texts.
Use relative units like rem and a unit for the padding, margins widths and font-size values. For more on CSS unit click here
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
Marked as helpful
1
Please log in to post a comment
Log in with GitHub
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