Francisco Carrillo• 5,540
@frank-itachi
Posted
Hello there 👋. You did a good job!
I have some suggestions about your code that might interest you.
HTML 📄:
- Wrap the page's whole main content in the
<main>tag. - If your code has different sections that have a specific purpose like a navigation, article, sections or footer, it’s a good practice to enclose those parts with HTML5 landmarks. For example, you could use a
<footer>tag to wrap the<div class=”attribution”>section. - Since the mobile design has a different image, you can use the
<picture>tag that allows you to interchange the images depending of the viewport size. Red more about this awesome tag here. - The heading order is important in the html structure so try to always start your headings with an
<h1>tag and then you can decrease by one if you need to use more heading in your html code.
I hope you find it useful! 😄 Above all, the solution you submitted is great!
Happy <coding /> 😎!
Marked as helpful
0
@Pinnacle007
Posted
@frank-itachi refering to the <main> tag, is it housing the entire content, including the picture
0
Francisco Carrillo• 5,540
@frank-itachi
Posted
@Pinnacle007 In this case yes because the whole card is the most important content in the website, so the card should be wrapped inside the main tag.
Marked as helpful
1
