pure HTML CSS & JAVASCRIPT

Community feedback

• 

  Joanna•370

  @JoannaLapaPosted over 2 years ago

  Hi Sudarshan!

  My congrats on finished project.

  The desktop design looks well and everything works well.

  My recommendation is to correct the mobile design for 375px - your card dimansions are bigger than the screen.

  Good luck!

  Marked as helpful

  1

  ---

  

  sudarshan hosalli•510

  @sudarshan161219Posted over 2 years ago

  @JoannaLapa thank you

  0
• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  Hi Sudarshan!

  Some recommendations:

  • To the .buttons class add the attribute aspect-ratio: 1 so that it takes a size equal is its height and width and shape is circular.
  • If you want the buttons to be centered I recommend you to place the following attributes to the .rating-buttons class:

  display: flex;
  flex-direction: row;
  justify-content: space-between;
  margin-top: 1em;
  

  And to the buttons that have the .buttons class, remove the margin-top and margin-rigtht.

  • To the .submit-btn class add the attribute:

  text-transform: uppercase;
  letter-spacing: 0.20rem;
  and change the width attribute to:
  width: 100%;
  

  • If you can change the div to footer of the box that has the class .attribution.

  With this you can improve your design, good luck!

  Marked as helpful

  1
• 

  sudarshan hosalli•510

  @sudarshan161219Posted over 2 years ago

  @Sdann26

  thanks for helpful feedback

  1