@MelvinAguilar
Posted
Hi @Roneeey ๐, good job completing this challenge, and welcome to the Frontend Mentor Community! ๐
Here are some suggestions you might consider:
1.Don't use <br>
, the <br>
tag is not semantic. It should never be used to add vertical spacing, it is only used in specific cases (e.g. poem or an address) also when a screen-reader reads the text it will break the text at the <br />
tag and break the flow of reading, You could use padding or margin styling via CSS to avoid them. More Information here.
For example, you could remove the <br> tag and add margin-inline: 1rem;
to the <p> element and you would have the same result.
2.Use margin: 0.938rem
or margin: 15px
in the .card
selector so that it has some space when viewed on mobile devices.
3.I see it well, although to confirm with the "Inspect Element" option of your browser you can see your result on other devices and get an idea of โโif it is centered.
I hope those tips will help you.
Good job, and happy coding!
Marked as helpful