All comments

• 

  Albina S•130

  @albina0104Submitted over 1 year ago

  

  Solution using semantic HTML, Sass, JavaScript

  1

  50
  

  Zozobrack•150

  @ZozobrackPosted over 1 year ago

  Overall, the code for the front-end mentor project looks good. Here are a few suggestions for improvement:

  HTML structure: Consider wrapping the form inside a section or a div with a class, instead of directly putting it inside the main tag. This will make it easier to style and manage.

  CSS: Consider using more descriptive class names instead of generic names like 'text' or 'card'. This will make it easier for someone else to understand your code and make changes later.

  Responsive design: The current CSS seems to be responsive.

  JavaScript: Good job on using event listeners and preventing the default form submission behavior. One suggestion is to add error handling in case the user doesn't select any rating before submitting the form.

  Overall, great job on the project and keep up the good work!

  1

  This comment has received 1 replyShow