Skip to content
OverviewSolutions
8
Comments
13
suraj gehlot
@enggsuraj

All comments

  • suraj gehlot•180
    @enggsurajPosted over 2 years ago

    A little bit of scaling and adjusting margin padding needs to be done. Other than that all is good. 😊

    1
  • suraj gehlot•180
    @enggsurajPosted over 2 years ago

    You can use position absolute to center the card container instead of adding margin-top:8%

    0
  • suraj gehlot•180
    @enggsurajPosted over 2 years ago

    Add <q><q> tag for quotes to get "" commas.

    On load, you get "Most things are not as bad as you think they are." as default.

    but on event click "" double commas are missing.

    Marked as helpful
    0
  • Omar•20
    @Omyy27Submitted over 2 years ago

    I have a doubt, sometimes my request takes a long time to arrive, I understand that with an Async function I could solve it. But I don't know how to implement it in my solution with fetch.

    Advice generator with CSS Flexbox and Fetch Js

    2
    suraj gehlot•180
    @enggsurajPosted over 2 years ago

    You can use <q><q> in place of the h5 tag for quotes to get "" double commas as per requirement in a challenge.

    Marked as helpful
    0
  • suraj gehlot•180
    @enggsurajPosted over 2 years ago

    You can use the font provided in readme file.

    1
  • suraj gehlot•180
    @enggsurajPosted over 2 years ago

    Add

    .iwc-primary { width: 80%; }

    for some width to Payment button

    0
  • suraj gehlot•180
    @enggsurajPosted over 2 years ago

    Please valid include the image path of dice and pattern divider.

    For example:

    <img src="./images/icon-dice.svg" alt="dice"> <img src="./images/pattern-divider-desktop.svg" alt="pattern-divider">
    0
  • Raul Olvera•250
    @rolvera22Submitted over 2 years ago

    Thanks for passing by. Please let me know if you find anything that can help me to improve my code. Thank you very much.

    Page using HTML and CSS

    1
    suraj gehlot•180
    @enggsurajPosted over 2 years ago

    Only add footer attribution should be outside your main class='container'.

    Marked as helpful
    0
  • suraj gehlot•180
    @enggsurajPosted over 2 years ago

    Instead of adding margin-top:100px in body

    Add

    body { background: hsl(212, 45%, 89%); display: flex; /* margin-top: 100px; */ justify-content: center; align-items: center; height: 100vh; }

    To perfectly center the container 😊

    0
  • suraj gehlot•180
    @enggsurajPosted over 2 years ago

    Hi, your solution is perfect, not a great deal but scaling can be adjusted.

    0
  • suraj gehlot•180
    @enggsurajPosted over 2 years ago

    Pretty much a point solution. Keep it Up. Just add more margin-top to the 'Order Summary' title

    Marked as helpful
    1
  • suraj gehlot•180
    @enggsurajPosted over 2 years ago

    Good

    Marked as helpful
    0