gary9630 / teacozy Goto Github PK

Either you have to use padding/margin to push those boxes down, or you have to convert the location div into a flex box and make adjustments to center everything.

Overall met expectations.

You did a good job matching the website spec. Everything was well organized and setup properly.

Your focus should be more gear to best practices in organization and efficiency.

The location div positioning is minor but it needed to be pointed out.

I also typically looked for unneeded css which is why pointed out the flex wrap usage.

As a challenge use media queries to make this site look like a mobile app on a mobile device.

This will help prepare you for the upcoming projects as they will be having use media queries quite frequently.

Great that you are using them in the html, to make things easier to line up, have matching comments in the CSS file.

.location-container{
display: flex;
justify-content: center;
flex-wrap: wrap;
}

We don't need flex-wrap in this container. If you use the chrome tools to remove it, you will see nothing changed.

Now on the other hand if you remove it from the tea container everything breaks. So the idea here is if you are expecting to have enough items to need to wrap them then use it.

In general unneeded code in our css makes it look like we are just slapping things on to see what works so better to clean this up and double check your css after you visually reached your goal.