Chanda• 800
@Chanda-Abdul
Posted
Good job on your NFT card component solution!
A few sugesstions
-
consider using less
<div>
's for html semantic purposes.<div>
's should only be used for styling purposes and not as containers.-
This article expains this concept well An intro to semantic HTML
-
ex. instead of
<div class="container">
just use<main>
-
instead of
<div class="price"><h2>
use<h2 class="price">
-
-
the active states are not working, you could add something like this
a:hover {
color: white;
cursor: pointer;
}
- define
font-family
in thebody
selector
body {
font-family: 'Outfit', sans-serif;
}
1