@brunomoleta
Posted
"Hola" John,
I advise you to avoid styling the `````body```` tag. Things may get unnecessarily messy doing so.
For starter, the body ought to have a width:100%
and then, inside the header, main and ````````footertag you may use adivwith the.max-w-[1440px]````.
Also, I don't see the need to use a section inside the header tag.
You can use an a,nav and button, because the header is the section, you know?
Another observation:
The article card will lead the user to another webpage,
so this shall not be a div tag, for divs have no semantic meaning.
It's better if this tag is an article.
According to MDN: The <article> HTML element represents a self-contained composition in a document, page, application, or site, which may be independently reusable. Examples include a forum post, a magazine or newspaper article, or a blog entry
Also, I recommend this blog: JoshComeau. He writes thoroughly about front-end programming. It's worth checking from time to time.
(If you find this comment helpful, please mark it as helpful)
Congrats on the effort, John :)
Best regards from Brazil
Marked as helpful
@JohnPugh688
Posted
@brunomoleta Thanks for your feedback. I struggle to get the grasp of semantic html. Understanding when to use different tags will hopefully come with experience. What you have recommended makes complete sense now I look at it so thanks for taking the time to have a look at my coding. Much appreciated 👍
