@ratul0407
Posted
@ThornenSan congrats on completing this challenge🎉🎉
You've done a really great job✨. The solution is really great but one thing that caught my eye is that you've used a div
tag for almost everything. Whenever you needed a container you used a div
. There is nothing wrong with divs
they are ideal container. But you should work a little bit on landmarks as they provide a great value for accessibility. You could've used a main
as a background container. You could've used a footer
for the .attribute
div. This will help you to cope with accessibility along the way.
These are little nick picks and instead of that the solution is great👍
Have a very nice day and keep up the good work🎉😄😄
Marked as helpful
@ThornenSan
Posted
@ratul0407 thank you for the feedback as always