@M-ilen
Submitted
@enggsuraj
@M-ilen
Submitted
@enggsuraj
Posted
A little bit of scaling and adjusting margin padding needs to be done. Other than that all is good. 😊
@enggsuraj
Posted
You can use position absolute to center the card container instead of adding margin-top:8%
@mritul
Submitted
@enggsuraj
Posted
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
@Omyy27
Submitted
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.
@enggsuraj
Posted
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
@Ivan96318
Submitted
@enggsuraj
Posted
You can use the font provided in readme file.
@Sayrarh
Submitted
@enggsuraj
Posted
Add
.iwc-primary { width: 80%; }
for some width to Payment button
@Ai7Ga
Submitted
@enggsuraj
Posted
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">@rolvera22
Submitted
Thanks for passing by. Please let me know if you find anything that can help me to improve my code. Thank you very much.
@enggsuraj
Posted
Only add footer attribution should be outside your main class='container'.
Marked as helpful
@CEE-AI
Submitted
Redone as suggested, Thank you @denielden and @enggsuraj. I can even get better... more suggestions are appreciated, thanks again.
@enggsuraj
Posted
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 😊
@allencode11
Submitted
@enggsuraj
Posted
Hi, your solution is perfect, not a great deal but scaling can be adjusted.
@DorisSch15
Submitted
@enggsuraj
Posted
Pretty much a point solution. Keep it Up. Just add more margin-top to the 'Order Summary' title
Marked as helpful
@hendsaeedd
Submitted
my challenge
@enggsuraj
Posted
Good
Marked as helpful