product preview card component responsive design

Community feedback

• 

  NicoR23•120

  @NicoR23Posted almost 2 years ago

  Hi, it looks really good. I'm just starting out with this, so I'm not sure if what I'm going to say is correct. In the <main> element, you used display: flex to align it in the center, but you also used flex-direction: column; with align-items: center; and I think you should remove the flex-direction; and just leave the align-items, or you can remove align-items and put justify-content: center;. I'm sorry if I'm wrong.

  Marked as helpful

  0

  ---

  

  Thomas B•160

  @TomBirbeckPosted almost 2 years ago

  @NicoR23 Thank you for the feedback, it was very helpful. I had a look back at my solution and I was actually able to completely remove display, flex-direction, and align-items from the main component at that point in the CSS file.

  Thanks again!

  0