KEERTHI B• 20
@keerthi797
Submitted
All feedback is welcome
Khalid Khabouz
@KhalidKhabouz
All comments
- Khalid Khabouz• 190
@KhalidKhabouz
Posted
Nice job, just try to use semantic elements as a habit for the long term, to help the search engines and other user devices to determine the importance and context of web pages.
Generally you did a good job keep up.
Marked as helpful
0 - Oussama Taali• 250
@oussamataali97
Submitted
Khalid Khabouz• 190@KhalidKhabouz
Posted
The layout looks nice, good job for this one. But I just have some remarks to improve the accessibility issues.
- Try to wrap your section element inside the main tag.
- Try to wrap the attribution div inside a footer tag.
Also to improve the typography of your design, use the font provided in the style-guide.md file.
Good job and happy coding
Marked as helpful
0 - Khalid Khabouz• 190
@KhalidKhabouz
Submitted
I've just completed this challenge, it was helpful. Thanks for stopping by.
Khalid Khabouz• 190@KhalidKhabouz
Posted
That was so informative and helpful, you are right, it is better to build good practices as a foundation. Thank you for your time.
0 - Khalid Khabouz• 190
@KhalidKhabouz
Submitted
This is my solution for this challenge. It was a good thing to work with multiple background images using CSS. Thank you for stopping by.
Khalid Khabouz• 190@KhalidKhabouz
Posted
Thank you so much for all these remarks, it was very helpful for me to build and develop a good practice. Thanks again for your time for stopping by.
1
