
Design comparison
Solution retrospective
This is the best challenge that i ever did, so thanks frontend mentor for giving us this opportunity.
Community feedback
- @tathanz98Posted 2 months ago
HTML Semántico ¿La solución incluye HTML semántico? El código HTML tiene una buena base, pero se podría mejorar el uso de etiquetas semánticas. Por ejemplo, en lugar de usar <div> para contenedores generales, sería beneficioso usar etiquetas como <section>, <article> o <header> cuando sea apropiado. Esto mejoraría la accesibilidad y la estructura del documento, permitiendo que los usuarios y los motores de búsqueda comprendan mejor el contenido de la página.
Sugerencia: Considerar usar <section> para las tarjetas y <header> para la parte superior del contenido, donde se encuentra el título y la descripción general. 2. Accesibilidad ¿Es accesible y qué mejoras se podrían realizar? Actualmente, la solución no parece incluir atributos importantes para la accesibilidad, como los atributos alt para imágenes y roles de ARIA. Aunque las imágenes tienen texto alternativo (alt), sería importante agregar descripciones más detalladas y específicas para mejorar la comprensión del contenido por parte de los lectores de pantalla.
Sugerencia: Asegurarse de que todas las imágenes tengan descripciones precisas y que los elementos interactivos, como los botones (si los hubiera en el futuro), tengan un enfoque accesible y roles definidos para facilitar la navegación con teclado o tecnologías de asistencia. 3. Diseño Responsivo ¿El diseño se ve bien en una variedad de tamaños de pantalla? El diseño parece funcionar bien en pantallas de diferentes tamaños debido al uso de media queries para manejar el comportamiento en pantallas pequeñas. Sin embargo, las tarjetas están muy apretadas en pantallas pequeñas (como en dispositivos móviles). Algunas tarjetas se superponen ligeramente o pueden quedar demasiado grandes para el espacio disponible.
Sugerencia: Se podría considerar agregar un ajuste de flex-wrap para que las tarjetas se distribuyan más adecuadamente en pantallas muy pequeñas. Además, los márgenes podrían ajustarse dinámicamente según el tamaño de pantalla. 4. Estructura, Legibilidad y Reutilización del Código ¿El código está bien estructurado, es legible y reutilizable? El código tiene una estructura clara, y los comentarios en el código ayudan a comprender el propósito de cada bloque. Sin embargo, sería bueno modularizar más los estilos, especialmente en los estilos repetidos para las tarjetas, como los bordes, sombras y tamaños de márgenes. Usar clases comunes o un sistema de variables podría mejorar la reutilización.
Sugerencia: Considere el uso de variables CSS para estilos repetitivos y la creación de clases de utilidad, lo que haría que el código fuera más limpio y fácil de mantener. 5. Diferencias con el Diseño ¿La solución difiere considerablemente del diseño? La solución es bastante fiel al diseño original en términos de la disposición y el estilo visual. Sin embargo, el uso de margin-top en las tarjetas para separarlas podría mejorarse, ya que podría hacer que el diseño se vea menos coherente en pantallas pequeñas. Asegurarse de que todos los elementos tengan márgenes o rellenos consistentes para mantener la uniformidad visual es clave.
Sugerencia: Asegúrese de que el espaciado entre las tarjetas se ajuste dinámicamente para garantizar que se vea bien en todas las resoluciones. Resumen de comentarios: HTML semántico: Mejorar el uso de etiquetas semánticas. Accesibilidad: Añadir descripciones más detalladas para las imágenes y asegurarse de que todos los elementos interactivos sean accesibles. Diseño responsivo: Asegurar una distribución adecuada de las tarjetas en pantallas pequeñas. Código: Mejorar la reutilización y modularidad, utilizando variables y clases comunes. Coherencia con el diseño: Ajustar los márgenes y espaciado para que el diseño se vea mejor en todos los tamaños de pantalla.
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