interactive-rating-component

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  👾Hello Moeka, congratulations on your new solution!

  I’ve some suggestions for you:

  To improve the card overall responsibility, you can start to add flex-wrap inside the class that manages the section for the rating numbers button and makes the adjust to fit in different rows while the container scales down, not that without this property the container doesn't shrink. Here's the code applying these changes:

  .ratingbtn-wrapper {
      display: flex;
      margin: 1.5rem 0;
      justify-content: space-between;
      flex-wrap: wrap;
  }
  

  ✌️ I hope this helps you and happy coding!

  Marked as helpful

  1

  ---

  

  rizkyizh•230

  @moe-kaPosted about 2 years ago

  😊Thanks. I have modified it 👍

  1
• 

  Rene•100

  @ProgrammerOfDeathPosted about 2 years ago

  Hey, nice work!

  I would suggest to remove the second main tag and move the content of the both cards into a more reusable card component, cause the two cards share same styles and so on.

  So there wouldn't be the sematic html error.

  Happy coding!

  Marked as helpful

  0

  ---

  

  rizkyizh•230

  @moe-kaPosted about 2 years ago

  🤗thank you @ProgrammerOfDeath. helpful solution. I will modify it

  1

  

  Rene•100

  @ProgrammerOfDeathPosted about 2 years ago

  @moe-ka No problem, I'm happy to help. I'll do the same challenge soon with Vue, so I can have a look on your solution. :)

  1