@Mar1362
Posted
hi, hope you are doing well! I really appreciate the way you did the challenge as well in the design than the way you coded it. I found it clear but i think that in your html code you may use <main> tag instead of <div class="container"> and i don't understant why you put 120vw for your background-size and 880% at the media one. Good continuation!
Marked as helpful
@0xabdulkhalid
Posted
REPLY:
- Thanks for leaving a comment which helps to improve my code, I' fixed the issue.
- Now check the project once again to explore the difference between old one.
@Mar1362
Posted
@0xAbdulKhalid yes i saw the code and it looks clean for me but still something in the html one, instead of doing this:
<div class="container" role="main">
...
</div>
you may do the following:
<main>
...
</main>
i didn't know about the difference since the visual result is the same and i did the same in my older codes but it is highly recommended to get the habbit of trying to use the right html segmentation in order to be comfortable with in more serious project later. I think you do not need to change anything in your html right now since it will may cause some changes in your css too (and i am really lazy :)) but you have better consider them in your next projects. checkout this link
Marked as helpful
@0xabdulkhalid
Posted
@Mar1362 Sure brother hereafter i will try to use semantic tags..:-)
@jmdonizetti
Posted
@0xAbdulKhalid Ficou muito bom, fundo ficou perfeito parabens
@0xabdulkhalid
Posted
@jmdonizetti
- commenting in english will help every people to understand brother
- Nice to hear your feedback, Thanks brother..
