All comments

• 

  NaimurRahman00• 180

  @NaimurRahman00

  Submitted 12 months ago

  Hello, I’m Naimur and this is my solution.

  Can you review my solution and any suggestions for me how I can improve?

  Thank you.

  

  Result summary component [Naimur]

  1

  40

  

  Malia Badziukh• 20

  @maliabadziukh

  Posted 12 months ago

  Hi Naimur, your solution looks great!

  A few pointers:

  • You did not include an assets folder in your project directory, so the icons throw an error since the file cannot be found.
  • You can use display: flex for the <section> (justify and align center) in order to center your component on the page. Now it's a bit off-center to the bottom.
  • Since you are using flexbox for the card layout, you can also use flex: 50% for both left and right cards instead of specifying their width in rem.

  I am relatively new to front-end, so test out / research my suggestions before implementing them, but I think they would be nice improvements :)

  1

  This comment has received 1 replyShow