Melvin Aguilar π§π»βπ»β’ 61,220
@MelvinAguilar
Posted
Hi @gdemartino93 π, good job completing this challenge, and welcome to the Frontend Mentor Community! π
I have some suggestions you might consider to improve your code:
- Use the
<main>
tag to wrap all the main content in your solution instead of using<div class="carta">
.
- Use
<footer>
instead of<div class="attribution">
. The<footer>
element contains authorship information.
- To make alternative texts more worthwhile, add descriptive text to the alt attribute of the QR image to explain what the QR image does. Upon scanning the QR code, you will be redirected to the frontendmentor.io website, so an example of alternative text would be "QR code to frontendmentor.io". You can read more about alternative text here.
- It might be more efficient to use the <p> tag instead of <span>; the <p> element represents paragraph-level content, usually text:
<p class="text-center text-secondary mb-3"> Scan the QR code to visit Frontend Mentor and take your coding skills to the next level</p>
- Use an h1 tag for your solution. The
<h1>
element is the main heading on a webpage, also, there should only be one<h1>
tag per page. Swap the<h2>
tag with<h1>
.
<h1 class="text-center fw-bold fs-5 mb-3"> Improve your front-end skills by building projects</h1>
Above all, the project is done wellπ. I hope those tips will help you! π
Good job, and happy coding! π
1
Gianluca De Martinoβ’ 100
@gdemartino93
Posted
@MelvinAguilar thanks for the tips but h1/2 has some default property from bootstrap and text wasnt the same.
0