News home page using HTML Css And Javascript

Solution retrospective
any type of feedback is welcomed
Please log in to post a comment
Log in with GitHubCommunity feedback
- @KarimAyman97
hi Shivam singh gautam great work . i have some recommendations for you
1- try to use separate file for CSS styling * for example *
<head><link rel="stylesheet" href="styles.css" /></head>
this will help improving code readability2- Add descriptive
alt
attributes for your images to improve accessibility.<img src="assets/images/logo.svg" alt="Frontend Mentor Logo" class="logo">
3- Make sure your navigation links have actual URLs. Currently, your <a> tags in the navigation are empty.
<li><a href="#">Home</a></li>
I hope it helps!
Other than that, great job!
Marked as helpful
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