@Agnik7
Posted
Hi, Congratulations on solving this challenge!!!๐๐๐๐ I just went through your code, and have a few suggestions that I feel will help you.
-
Replace the
h3
tag with anh1
tag to get rid of the accessibility issue. -
In the CSS file, you don't have to mention
width:100%
to thebody
tag, because, for any element, if width is not mentioned, it takes up a 100% of its width by default. -
In the
body
tag, replace theheight:100vh
withmin-height:100vh
, for better responsiveness. -
For the
p
tag inside thediv class="value"
tag, instead of using font-weight, you can use the <b></b> to make the text bold, thereby requiring one less CSS line.<p><b>0.041 ETH</b></p>
Hope this comment helps you. Please feel free to correct me if I said anything incorrect. Have a nice day!!
Marked as helpful