@rowanDeveloper
Posted
Hi @jeffgicharu,
Congrats on submitting your first solution! Keep it up! FrontEnd Mentor is a great place to practice and improve your coding skills.
I took a look at your code and I like the fact that it is pretty clean and concise. I would add some changes, like, for example, using the <main> tag instead of <article> or <section>. The idea is that the <main> tag holds everything. I would also have a div container, just so I could better manipulate the padding and another div for the image-container to better adjust the width and height of the image. But that is just my logic of programming.
Something like:
body
<main>
<div class="container">
<div class="image-container">
<img href="./">
</div>
code
</div>
</main>
Overall, congrats on your first project!
Marked as helpful
@jeffgicharu
Posted
Hello @rowanDeveloper Thank you very much for your feedback. I started learning HTML and CSS recently and the teacher that I'm following on YouTube suggests avoiding using divs too much and instead insists on using specific HTML tags so that the code can have more meaning.