@SJ-Nosrat
Posted
Hi Jose, Great solution! Well done.
- With respect to semantic HTML I'd wrap everything inside your
<body>
with the<main>
tag. - I'd remove
width: 100vw;
from your.card
class and keep themax-width: 100%;
- Your use of CSS Flexbox is fine.
- I'd look into your card in the mobile view; the card is too wide and is not giving enough breathing room between the edge of the screen and the card; do you approach the challenge with a mobile first approach? The issue here might be:
width: 400px;
on the.container
class in your CSS. - I think removing
width: 400px;
from the.container
class and adding amargin: 0 1.5rem
would help, although it does squeeze some of your other content inside the card; I'd look into restyling those.
It can be hard coding the challenge just by eye-balling it. You're doing really well! Keep on coding and best of luck with your coding journey; looking forward to more of your solutions.
Cheers!
Once again well done
Marked as helpful
@jgbpessoa
Posted
@shahin1987
Thank you so much!
I completely forgot about the mobile view and the mobile first approach haha Thanks for reminding me!
I'm gonna update this solution with your suggestions and also add a better looking summary component for mobile screens. I just gotta say thank you again for taking your time to review my code.
Have a great rest of the weekend!
@SJ-Nosrat
Posted
@jgbpessoa My pleasure! Have a wonderful weekend too.
@vanzasetia
Posted
@shahin1987 I would recommend everything in your page content should be wrapped with main
tag, except the attribution. I will use footer
instead of div
for my attribution.
@SJ-Nosrat
Posted
@vanzasetia Didn't notice that; good catch!