Design comparison
Solution retrospective
Hello guys! Any feedback is welcome.
Community feedback
- @Sdann26Posted over 2 years ago
Este proyecto esta bien muy bien maquetado en lo personal te ha quedado bastante bien el responsive y todo.
Acá masomenos te mencionaría lo mismo que el otro proyecto revisar el reporte que generá frontendmentor para que salgan sin errores ya que siempre es bueno dejarlo en 0 errores porque la mayoría de los que te muestran son por usar algún atributo obsoleto, falta en los elementos para la accesibilidad, errores en el html, etc.
Para ir eliminando los errores cambia el
<div class="container">
por<main class="container">
. Tambien cambie<h2>10,000+ of our users love our products.</h2>
por<h1>10,000+ of our users love our products.</h1>
el motivo principal de este cambio es porque siempre se necesita mínimamente un h1 por página ya que es el titulo principal del proyecto los demás h2, h3, ... h6 funcionan como subtitulos al igual que en un informe, osea el h1 es el titulo, el h2 podrías ser subtitulos pos sección y el h3 subtitulos de cada sección de la sección que tiene el h2 y así sucesivamente. Por ultimo tienes que borrar unos atributos absolutos de tus etiquetas<link>
lo puedes ver en el reporte en la parte HTML Validations.Termina de modificar eso y al generar un nuevo reporte debería salirte 0 errores, espero que mis comentarios sean de ayuda :D!
Good Coding!
Marked as helpful1@MartinsitoBritoDiazPosted over 2 years ago@Sdann26 Gracias por tomarte el tiempo de revisar el codigo y brindarme tu conocimiento, eso lo aprecio mucho. Voy a mejorar el reporte implementando esas recomendaciones.
1
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