Interactive rating component

Community feedback

• 

  Eileen dangelo•1,600

  @Eileenpk

  Posted over 2 years ago

  Hi Nahuel-cell, your project looks good.

  I would consider changing the let rating; to let rating = null

  It is generally considered good practice to initialize a variable with a value, rather than declaring it without a value. If you are declaring a variable with a type "let" or "const" and you're going to assign it a value later, it's a good idea to initialize it with a default value, like null. This is because, if you don't initialize a variable and try to use it before assigning a value to it, it will throw a "ReferenceError: variable is not defined" error. By initializing a variable with a default value, you can avoid this issue and make your code more predictable. Additionally, if you are going to use a variable in an expression, it's a good idea to initialize it with a default value that is appropriate for the expression, such as an empty string or zero. So, it is considered a good practice to initialize a variable with a default value, such as null, when you are going to assign it a value later.

  Hope you found this helpful!

  • Let's connect on LinkedIn! - @Eileenpk

  11

  Marked as helpful
• 

  Dytoma•550

  @Dytoma

  Posted over 2 years ago

  Hey👋

  Good job on completing this challenge👏

  I have some feedback that you can use to improve your code:

  • First instead of using article elements, I will suggest you use div elements and give them a role attribute(eg: <div role="main">....</role>).
  • For the <div class="attribution">......</div> you can add a role="footer" you can learn more about landmarks here. And for <img src="./assets/images/illustration-thank-you.svg" alt="Illustration" class="article__illustration"> I will suggest you add an aria-hidden='true' attribute for screen readers to skip this image as it just serves for decoration.

  Happy coding🙌

  02

  Marked as helpful