@VCarames
Posted
Congrats @kushyzee on completing this challenge!
In your <body> element, you'll want to change your height
to min-height
for better responsiveness.
Next, since this a card, for better semantic HTML you want to change you <main> element to <article> element. You'll also want to change all your <section> elements to <div> instead, since the <section> element is used to define a section of a webpage.
For your images to change between mobile and desktop is better to use the <picture> element, it use less code and its far more effective.
I attached a link for more detail about it:
<picture>
<source media="(min-width: )" srcset="Desktop image goes here">
<img src="Mobile image goes here" alt="">
</picture>
@kushyzee
Posted
@vcarames wow, thank you so much for feedback. I totally forgot about the <picture>
element