@RichardOgujawa
Posted
@Teles23 No worries man, more than happy to help! Best of luck with coding going forward, I'm sure with an attitude like the one you have right now you'll do great things:)
@RichardOgujawa
Posted
@Teles23 No worries man, more than happy to help! Best of luck with coding going forward, I'm sure with an attitude like the one you have right now you'll do great things:)
@RichardOgujawa
Posted
Hi there,
Hope you're keeping well:) I really like your coding solution, especially the fact that you indented your code. It makes it easy to read. I appreciate that you used the different images instead of just scaling it down for the mobile version, didn't see many other people doing that, unless I wasn't looking well enough.
However, I would make some changes to the code. Then again these are just my opinion so feel free to take this on board or take it with a grain of salt, I wish you all the best either way.
In the HTML:
In the CSS:
Hope this helps:) And if you want me to explain anything a bit better just let me know, I was trying to keep this not too long, so I had to rush through some concepts.
@Teles23
Posted
@RichardOgujawa Primeiramente muito obrigado pelo feedback! Você trouxe os pontos que realmente eu tive dificuldade principalmente em como utilizar duas imagens diferentes para telas diferentes, por isso optei por usar como Background. Estarei corrigindo esses pontos e melhorando cada vez mais, muito obrigado!!
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord