Solution using HTML, SASS, BEM and CSS transitions.

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Nice job in completing this project. Regarding your solution, I would like to recommend the following for you…

  • Add border-bottom-left-radius: 8px; in .grid__container--2 and border-bottom-right-radius: 8px; in .grid__container--3 rule set.
  • Add the same properties in .grid__container--3 for the responsive design

  Above all, the project is done well. Keep up the good work and Hope this is helpful!

  Marked as helpful

  0

  ---

  

  Karol Binkowski•1,640

  @GrzywNPosted almost 3 years ago

  @RioCantre You're right, I completely forgot about this 😅. Solution is updated now. Have a nice day!

  1
• 

  Hexer•3,660

  @EmmanuelHexerPosted almost 3 years ago

  Great job overall. Well done

  Marked as helpful

  0
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  Hay ! Karol Good Job on challenge

  -> i believe that background color is given in style-guide so change it

  -> Check my article on accessibility issues

  If this comment helps you then pls mark it as helpful!

  Have a good day and keep coding 👍!

  0

  ---

  

  Karol Binkowski•1,640

  @GrzywNPosted almost 3 years ago

  I used colors and opacity properties from Figma, so they match the design. Have a nice day 👍!

  1

  

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  @GrzywN But why is it dark then the original design? did they change the design

  0

  

  Karol Binkowski•1,640

  @GrzywNPosted almost 3 years ago

  @Crazimonk You're correct, it was a typo in variables file. Now everything is fine.

  1