Melvin Aguilar 🧑🏻💻• 61,220
@MelvinAguilar
Posted
Hi @mbonamensa 👋, good job on completing this challenge! 🎉
I like this solution for the challenge. Here are a few suggestions I've made that you can consider in the future if you're looking to improve the solution further:
- The
<h1>element is the main heading on a web page. There should only be one<h1>tag per page. The HTML Section Heading elements (Reference). This case requires thatSUVS,SEDANS, andLUXURYbe<h2>elements. - The
Learn Moreelements should beanchortags and not buttons because they redirect to another part of the page. - Instead of using pixels in font size, use relative units of measure like
remorem. The font size in absolute length units (px) does not allow users with limited vision to change the text size in some browsers. Reference.
I hope those tips will help you.
Good job, and happy coding!
Marked as helpful
0
@mbonamensa
Posted
@MelvinAguilar These are really great tips! On the anchor tags and buttons, is it advisable to use both? As in wrap the buttons in the anchor tag or remove the button element entirely and style the anchor tag to look like buttons?
0
Melvin Aguilar 🧑🏻💻• 61,220
@MelvinAguilar
Posted
@mbonamensa Hi! 👋 You should use the anchor tag directly and style it as a button. :D
0
@mbonamensa
Posted
@MelvinAguilar Alright, thank you for your feedback!
1
