@MelvinAguilar
Posted
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- Instead of using two media queries to modify the width of the
main
element, it's more advisable to use a fixedmax-width
of 900px to prevent it from stretching too much. Additionally, set a margin of 0.5rem to avoid it touching the edges of the screen on mobile devices.
main {
/* width: 85vw; */
max-width: 900px;
margin: 0.5rem;
}
- You can use classes instead of using IDs and nth-child since they are less specific.
- You can enclose the price and text in a single paragraph to make it read continuously in screen readers:
<p class="price"><span class="white">$29</span> per month</p>
.
I hope you find it useful! ๐
Happy coding!
Marked as helpful
@teenabobeena
Posted
@MelvinAguilar thanks so much for the suggestions! I will keep them in mind for future projects.