Shanxx191919
@Shanxx191919All comments
- @eman2point0Submitted 5 days ago@Shanxx191919Posted about 4 hours ago
The HTML structure is good overall, using semantic elements like <header>, <main>, and <footer>. However, you could improve accessibility by adding more descriptive alt attributes to the images and using aria-labels or role attributes for better screen reader support. Consider adding more semantic tags like <article> or <section> for improved SEO and readability.
For responsiveness, the layout is flexible due to the use of flexbox, but you might want to add media queries to fine-tune the layout on smaller screens, especially adjusting the card width and text size. The code is clean and readable, but splitting the CSS into a separate file could improve maintainability, particularly for larger projects.
0 - @AbestianSubmitted about 21 hours agoWhat challenges did you encounter, and how did you overcome them?
The background image wasn't showing on live site. Correcting path to image inside url() from '/images/x' to '../images/x' fixed the issue.