@vanzasetia
Posted
Hi, BryanCarlos! π
I recommend adjusting the size of the cards. Based on the design comparison, the cards are bigger compared to the design. Try to make the site looks as close as possible to the design.
Replace all the <h1>
with <h2>
. There should not be more than one h1
on a page. Many <h1>
elements mean many titles which can confuse the users, especially the screen reader users.
The buttons do not need to be wrapped by a <div>
. You should remove them.
The "Learn more" buttons should not be <button>
elements. If you think that the users will get navigated to another page when they click one of those buttons, then you should use <a>
elements instead.
Never use px
unit for font sizes. Use rem
or em
instead. Relative units such as rem
and em
can adapt when the users change the browser's font size setting. Learn more β Why you should never use px to set font-size in CSS
I hope this helps. Happy coding! π
Marked as helpful