HTML,CSS BOOTSTRAP JAVASCRIPT

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  👾Hello @Damilarr, Congratulations on completing this challenge!

  One tip about responsiveness:

  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:

  .d-flex {
      flex-wrap: wrap;
      display: -ms-flexbox!important;
      display: flex!important;
  }
  

  ✌️ I hope this helps you and happy coding!

  Marked as helpful

  0

  ---

  

  Damilarr•40

  @DamilarrPosted about 2 years ago

  @correlucas Oh...I didn't think about that Thanks 👍

  0
• 

  Hyron•5,870

  @hyrongennikePosted about 2 years ago

  Hi @Damilarr,

  Congrats on completing the challenge

  Instead of the browser alert box which is bad for user experience, I would suggest just disabling the button until a rating is selected this can be done by adding the disabled attribute on the button and removing it when the rating is clicked

  Hope this is helpful

  Marked as helpful

  0

  ---

  

  Damilarr•40

  @DamilarrPosted about 2 years ago

  @hyrongennike okay.. thanks

  0
• 

  Kawsar Ahmed Fahad•2,660

  @faha1999Posted about 2 years ago

  Hello, Damilarr Congratulations on finishing this project. It's lovely and great on the whole! Just a little tip:

  • You might want to use semantic tags like the <main> to wrap your code, instead of div. like

  <main class="bdy d-flex justify-content-center align-items-center flex-column">
  </main>
  

  You could also make use of <section> and/or <article> semantic tags. This would help improve accessibility.

  I hope it will work. Happy coding.

  Marked as helpful

  0

  ---

  

  Damilarr•40

  @DamilarrPosted about 2 years ago

  @faha1999 Alright thanks for the tip. I'd try it out

  0