David• 8,000
@DavidMorgade
Posted
Hello Achinta, congrats on finishing the challenge!
I would suggest you to set the background pattern as a brackground-img in your CSS instead of putting it as an html img, in this type of cases it fits perfectly to it in your CSS files instead of your html.
Also try to use more semantic tags, you could put all of your content inside a main tag, instead of just using a div, you could also use section tag to divide the parts of your card that are semantically related!
Hope my feedback helps you for future projects, good job!
Marked as helpful
0
Achinta Haldar• 140
@FSwebdeveloper
Posted
@DavidMorgade thanks you for your advice. This advice really helpful for me.
1
