@Da-vi-de
Posted
Hi, i get a sense you put all your effort to accomplish this challenge, i compliment to you, the result is nice!
-
Without seeing your code first and just moving back and forth a couple of times the window in the console i figured out this was not a mobile first responsive website. Then i saw the code... I want to encourage you to change approach and start the mobile version, trust me you won't need 1000 lines of css code the next time.
-
I also suggest to not underestimate the report, for each html issue there's a link, you can read what that is about and learn important things that help you to evolve and improve as a developer.
-
The html file is hard to follow, nearly unreadable; it's better writing some text as a comment rather than numbers and leave some more space. I tell you what i do but it's not a mandatory way of working. Usually when i have a huge block of code in which i have many nested
<div></div>
elements and those divs are just for styling i tend to leave a space, up to an important block of code like so:
<div class="...">
<div class="...">
<div class="...">
<article>
<h2>.....</h2>
<h2>.....</h2>
<p>.......</p>
</article>
</div>
</div>
</div>
so that if someone needs to take a look at your code can easily understand what is what!
Hope it helps, keep coding :-)
Marked as helpful