@omarmohy98
Posted
Good morning@sachdevavaibhav, wonderfully done! The code is remarkably responsive. Below is my feedback on potential improvements!
1.) Instead of <div class="main"> , let's replace that with <main> element and make it the top level in your code. Doing so will give your code more semantic meaning, will allow screen readers to know THIS is your main content and Fix the issues appear in Accessibility Report.
2.) Try to control images' width & height in style.css not in <img> element in your index.html.
If you found my feedback helpful, please mark this comment as helpful. Wonderfully done. Happy coding!
Marked as helpful
@sachdevavaibhav
Posted
@omarmohy22 Thank You for your feedback. I'll surely work on your suggestions and make the webpage more accessible.
@sachdevavaibhav
Posted
@omarmohy22 Hey! I have updated my solution. Can you please review it and give me feedback?
@omarmohy98
Posted
@sachdevavaibhav your solution now become very good but I suggest that you read more about semantic elements it will help you in your challenge solution.
Marked as helpful
@sachdevavaibhav
Posted
@omarmohy22 Thank You! I'll surely learn more about semantic html.😇
