@MelvinAguilar
Posted
Hello there π. Good job on completing the challenge !
I have other suggestions about your code that might interest you.
- Avoid using uppercase text in your HTML because screen readers will read it letter by letter. You can use the
text-transform
property to transform the text to uppercase in CSS.
- You should use the
<a>
tag instead of the<button>
tag because theLearn More
button is a link to another page. Use buttons to perform actions like submitting a form or closing a modal and use links to navigate to another page. You can read more about this here π.
- Avoid using percentage widths in the component; it can cause distortion on some devices. Instead, consider using
max-width
to adapt to the screen size and prevent excessive growth. Also, set a definedmargin
of 20px to avoid touching the screen edges.
- You don't need to add
font-family: 'Lexend Deca', sans-serif
to each<p>
separately. Instead, add it to body.
- The
<article>
tag is typically used to represent a self-contained piece of content that could be distributed and reused independently. If the content inside the<article>
tag is not a standalone piece, or if it doesn't represent an independent piece of content you might want to reconsider using the<article>
tag
I hope you find it useful! π
Happy coding!
Marked as helpful