Design comparison
SolutionDesign
Solution retrospective
Any feedback appreciated
Community feedback
- @GabrielLaminasPosted over 2 years ago
Hello, nice work. I prefer h1 to div in 'Order Summary' text. Because it's more semantic than div. I think if you add a div below the 'illustration hero' img and put the rest of the content inside it. This is nice to work with padding. Look at:
<div class="container__info"> <h1> Order Summary</h1> <p class="description"> You can now listen to millions of songs, audiobooks, and podcasts on any device anywhere you like! </p> <div class="price"> <img src="./images/icon-music.svg" alt="music icon"> <div class="price-center"> <p class="plan">Annual Plan</p> <p class="year-price"> $59.99/year</p> </div><a href="#" class="change"> Change</a></div>
<div class="payment"> <a href="#"> Proceed to Payment</a> </div><a href="#" class="cancel"> Cancel Order</a>
</div>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