Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive HTML5/CSS3/JS Interactive Rating Component

P

@EONRaider

Desktop design screenshot for the Interactive rating component coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I had a lot of fun in this project making use of localStorage for the first time to communicate state between different web pages on the same project. It also gave me the opportunity to practice more prototype-based implementations on JS so I can keep being the OOP maniac I like to be! :)

Community feedback

Daniel 🛸 44,230

@danielmrz-dev

Posted

Fala Alysson!

Seu projeto está excelente!

Tenho uma única sugestão:

  • Pra tornar seu código HTML mais semântico, use main pro conteúdo principal ao invés de uma div.

Essa mudança de tag não gera nenhum impacto visual mas torna o seu código HTML mais semântico e melhora a otimização SEO e accessibilidade do projeto.

Espero que ajude!

Fora isso, ótimo trabalho!

0

P

@EONRaider

Posted

@danielmrz-dev Opa Daniel. O que você menciona é fato. Fica melhor mesmo. A questão é que eu tenho usado a metodologia CUBE e ela sugere que eles elementos de composição não devam ser usados para estilizar. Assim sendo eu acabo usando a div externa para posicionamento e dimensões e o conteúdo mesmo entra no elemento main (esse sim contendo estilos e tudo mais, porém declarado como bloco).

No fim é mais uma questão de seguir a convenção neste caso.

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord