Looks good! A few comments I have: Have your editor align your code for you, there are a number of tools out there to help you with this, and it makes the code much more readable You define variables with let a lot rather than const in many scenarios where you could (and should) be using const . It's good practice, adding an extra layer of safety, among other things. This is arguable, but having separate presentation and container components makes code easier to read. You have a few large components, with a lot of JSX in the render function, which itself it complicated to read. Related to the above point, make many smaller, more composable components. It's a lot easier to understand (1) rather than (2) (1) < Product > < ProductImage img = {img} /> < ProductName name = {name} desc = {desc} /> </ Product > (2) < div className = "product" > < div className = "product__image" > < img src = {img} /> </ div > < div className = "product__name" > < h3 > {name} </ h3 > < p > {desc} </ p > </ div > </ div >

At least learning a lot of the gotchas such as this in Javascript (see here ) can help you sometimes avoid headaches when trying to debug code. But learning the basics and jumping to frameworks is probably fine.