Hi Adetuyi,
This looks pretty good and you've clearly done a lot of work here, but I think you could still get it closer to the design. Really try to pay attention to small details. Like spacing, sizing of elements relative to each other, width of borders, line spacing etc.
The html needs some work still. It's not accessible at the moment because you've used non-interactive elements like divs for click events (always use interactive html elements like buttons or anchor tags).
Document semantics are also missing because most text is in spans instead of meaningful html elements. That will make it very difficult for search engines and assistive tech to traverse your site. The page also needs a h1.
Looking at the javascript, it's OK and all functions fine, but it's using old methods. Try to leverage es6 in this - const or let, arrow functions etc. See if there are ways for you to Shorten and simplify the code.
It sounds like I'm listing lots of things, but they are all pretty easy to fix and fundamental to good practice. Spend extra time giving really good attention to detail and you'll notice the difference in your projects.
Good. Luck ☺
@Adetuyi
Posted
@grace-snow thanks, will work on it
