interactive-rating component

Community feedback

• 

  hamza•390

  @HamzaouladevPosted over 1 year ago

  hello @Hola-Code001, here's some feedback regarding your layout i hope you find interesting:

  • it's usually a bad idea to give your component a fixed height, your component should dynamically handle any large amount of content in any screen width without running into overflowing issues, its best to leave the height to a default auto.
  • a cool, simple way to handle spacing without worrying about unpredicted flexbox behavior is called The lobotomized owl selector, i invite you to check it out!

  i hope you found my feedback helpful, keep hacking!

  Marked as helpful

  0

  ---

  

  Hola-Code001•110

  @Hola-Code001Posted over 1 year ago

  @Hamzaouladev thanks

  0
• 

  Hussain Hussain•410

  @LysitheaDarkKnightPosted over 1 year ago

  Hello, how are you? Good job on completing the challenge!

  To combine your wrapper width and max-width, just write min(90%, 400px) for width! It'll automatically resize your element depending on the screen size by using whichever can be the smallest! It'll save you 2 lines of code so I recommend you try it out!

  Happy coding!

  0