Design comparison
Solution retrospective
refresqué la pantalla una vez enviada el valor por innerHtml no sé si será la mejor forma.
Community feedback
- @abrahamemamaniPosted almost 2 years ago
Hola, fersrm! La solución no está mal, pero quizá en un caso así podría ser mejor tener todo el contenido en el html y ocultar/mostrar el contenido que corresponda. Y desde el script sólo hacer ese cambio, agregando o quitando una clase.
Recomendaciones:
- Intenta escribir el código siempre en el mismo idioma (ideal inglés).
- Cuando las cosas se vuelven reutilizables no es muy aconsejable trabajar con el atributo id en las etiquetas html.
- Que el estilo y el comportamiento no estén atados. Si usas la clase .card para estilos, usa .js-card en el script. Y agrega ambas clases en la misma etiqueta html.
Marked as helpful0@fersrmPosted almost 2 years ago@abrahamemamani hola muchas gracias por su comentario. lo del segundo punto a lo de cuando las cosas se vuelven reutilizables y usar el atributo id eso si no entendí a qué se refiere?
0@abrahamemamaniPosted almost 2 years ago@fersrm Buenas! me refiero a que en un documento html el id de las etiquetas tiene que ser único, no se deben repetir, porque puede generar comportamientos inesperados. Y lo más conveniente en este caso es no utilizar id's, y usar clases para seleccionar el elemento desde los scripts, en lugar de hacerlo por el id.
0
Please log in to post a comment
Log in with GitHubJoin 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