Skip to content
OverviewSolutions
5
Comments
3
Gabriel de Freitas Laminas
@GabrielLaminas

All comments

  • Gabriel de Freitas Laminas•140
    @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
  • Gabriel de Freitas Laminas•140
    @GabrielLaminasPosted almost 3 years ago

    Thanks for your feedback it's so important for me. I am going to watch this video.

    0
  • Gabriel de Freitas Laminas•140
    @GabrielLaminasPosted almost 3 years ago

    Hello there. In the main content the button view plan is not click because a bg-pattern-intro-left-desktop is over it and the same problem is happening how we work button with bg-pattern-how-we-work-desktop over it. I think you must add z-index in these buttons and check them in diferents breakpoints.

    Marked as helpful
    0