This looks like a great solution to me. Don't stress so much!
Only very minor suggestions from me:
- Consider adding some space between footer and the card. They shouldn't be touching like I see on my phone
- The space between the component and screen edges is really tiny on my phone. Consider increasing that slightly
- It's a good habit to get into ASAP to include a modern css reset at the start of the styles in every project
- There is no need for a media query in this. Don't add unnecessary bloat
- Similarly, there is no need for a div wrapping the image
- Try not to make image alt text too verbose. This only needs to say QR code to FrontendMentor.io.
Marked as helpful
@LeonCelestino
Posted
Hello @grace-snow! Thank you very much about the feedback. Aside from the 3rd suggestion, I tried to match what you said.
If it doesn't bother you, would you check if it is alright now? I would be very pleased with any more suggestions and tips!
@LeonCelestino looks good. I'd consider using classes instead of element selectors in css and have a think about why you've used h3