
Solution retrospective
Any feedbacks are welcome...
Please log in to post a comment
Log in with GitHubCommunity feedback
- @MarziaJalili
Nice and neat! 🎉
🌟 A tiny little tweak?
✅ Including words like "Published" inside a <time> element is not best practice because this element is meant to wrap only the date/time content, dear.
✅ Here's the fix:
<div> <span>Published </span> <time datetime="2025-05-19">19 May 2025</time> </div>
Great work overall; may Allah bless you with more! ☺️
- @Md-Mubin
So far so good. Your previous projects are also great. You can add, loading="lazy" in img tag. Makes page faster and boosts performance on mobile. For this small one does not matter that much. But in big big project it does.
You can write responsive codes in another file. And connect it with the html. Like create a responsive.css file and than connect it with html just like the style.css. you can create a css folder and put the style and responsive css file in it. Good for the folder/file structure.
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