HTML, CSC FLEX

Community feedback

• 

  Michael Bishop•1,080

  @MikeBish13Posted over 3 years ago

  Hey, great job on the project.

  One major thing I've noticed:

  • When you get to around 780px screen size, the end of the content box gets cut off. This is because you have a container with a width of 70% and overflow: hidden set, so when the content is too big for the container it disappears. You could solve this by either making the breakpoint of your media query larger, or perhaps explore another way of centering your content box without the need for a container -- flexbox could be a neat way of doing this.

  On a more general note, you have exactly the same styles repeated for each of the three sections in your container. You could maybe consider applying a single class to each of the three sections, eg class="car-container", and that would drastically reduce the amount of code that you have to write.

  Hope that helps!

  1

  ---

  

  Elizabeth Okuta•150

  @MimieveshofuoPosted over 3 years ago

  @MikeBish13 Thanks for the feedback.

  0