@R3ygoski
Posted
Hello again Kerwin, congratulations on completing another challenge!
Following up on the comment below (or above), regarding the <article>
, it's used for self-explanatory content, meaning it doesn't depend on the page as context. Using it for the card is correct, but you used two <article>
elements, and one of them is in the wrong context. I'll explain below:
<article class="container-articles">
, the correct tag for this would be<main>
, as it contains all the main content of your page, which in this case is the card with the information.
Regarding the <button>
elements, it would be more appropriate to use <a>
tags because the idea is that clicking on them should redirect you to a new page. Therefore, instead of just using a <footer>
, it would also be a <nav>
(in this case, <nav>
within <footer>
).
Edit: I said the right is to use <nav>
, but actually the correct thing is to use <ul>
with <li>
, because <nav>
is dedicated to navigation within the page itself.
A bonus tip is that you could also use an <address>
tag for <p class="location">
, as it makes more sense since it's related to the location.
Congratulations again, your project is very semantic. Keep practicing and improving, you're doing great. If you need help with anything, please ask below.