Perfect Responsive Four card Feature Using CSS Flexbox.

Community feedback

• 

  Adarsh Pratap•5,515

  @adarshcodesPosted over 4 years ago

  Hi! @Thomas-pop, Congratulations on submitting your first solution👍 and welcome to this community. Your responsiveness is working fine on smaller devices. Let's see your solution from different aspects:

  • Design aspect: your design looks fine, one thing you can improve is the color of the border-top for the cards are quite dark make it a light color, you can find the color in the style-guides file.
  • Responsive aspect: Your solution is well responsive for the devices, you can add a two-column two-row layout of the cards for the tablet devices around 980px-700px.
  • Code aspect: Code looks clean but you can add a common class to add the same styles for each card so that your code will be DRY,(like for Karma, Calculator, Builder, Supervisor you've used similar design so make a common class something like cards and add that design to that class).
  • Issues aroused: Great work👏 it's your first solution and no issue aroused. Happy Coding🙂

  3

  ---

  

  Thomas pop•40

  @Thomas-popPosted over 4 years ago

  @adarshcodes,Thanks buddy.

  0