generated from microverseinc/readme-template
-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
Great work on this project 😍! Kindly take a look at the following comments😁
- Great work with your date functionality 😎, however if you want other ways of implementing it, you can check out the
toLocaleDateStringmethod of the date class. - I would like you to consider deleting the
atags in thenav-item's because they are unnecessary for this page. - Kindly consider removing the
default datein thedate-containerdiv since you are generating it dynamically with JavaScript. - Additionally, you should put the
h1tag before any otherhxtag that you have on your file, in order to follow theHTML semantics. - Your approach to removing the class in the books
div's with JavaScript looks great, but you could do the same withCSSand thenth:child(odd). This will select only theoddchild in the class.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels