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
Desktop design screenshot for the Blog preview card coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Raptor0x1 210

@Raptor0x1

Posted

Here are some things you should work on.

  • Your font isn't working. Remove the quotation marks on the sans-serif. Fallbacks fonts types such as serif, sans-serif, roboto etc don't require quotation marks. Font property cannot be used with single value it should have at least two values of font-size and font-family. So better to use font-family. Reference

  • If you open it in the dev tools and bring your cursor over the content class you can see there is some space below image. This is the default space set by the browser for the inline element. To get rid of that space use display: block;.

  • There was no need to use .card::before instead you can use box-shadow to gain this effect.

  • Use 'rem' or 'em' instead of 'px'.

0

@thanhsHai

Posted

Semantic HTML Your solution does a good job incorporating semantic HTML elements, which greatly enhances the readability and accessibility of the markup. Using elements like <header>, <nav>, <main>, and <footer> appropriately helps structure the content meaningfully.

Accessibility The solution shows attention to accessibility, but there are some areas for improvement:

Ensure all interactive elements (like buttons and links) are keyboard accessible and have visible focus states. Use aria-* attributes to improve screen reader support, especially for dynamic content. Include alt attributes for all images to provide descriptive text for screen readers. Consider using landmarks (like <section>, <article>, etc.) to define regions of the page more clearly. Responsive Design The layout adapts well to various screen sizes, providing a good user experience across devices. Here are a few suggestions for further improvement:

Test the design on more breakpoints to ensure it looks great on all device sizes. Ensure text and interactive elements are appropriately sized on smaller screens to maintain readability and usability. Consider using relative units (like em, rem, percentages) instead of fixed units (px) for better scalability. Code Quality Your code is well-structured and readable, which makes it easy to understand and maintain. Here are a few tips to enhance reusability and maintainability:

Break down large components into smaller, reusable ones. Use descriptive naming conventions for classes, IDs, and functions. Maintain consistent formatting and indentation throughout the code. Add comments to explain complex logic or sections that might not be immediately clear. Design Consistency The solution closely follows the design provided, which is great to see. Here are some minor points to consider:

Ensure color schemes, fonts, and spacing match the design specifications exactly. Pay attention to the details like border-radius, box shadows, and other subtle design elements to achieve a polished look. Regularly compare the implementation with the design to catch any deviations early. Overall You have done an excellent job with this solution, addressing key aspects of semantic HTML, accessibility, responsive design, and code quality. With some minor improvements, it could be even better. Keep up the great work!

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