Design comparison
SolutionDesign
Community feedback
- @michel-moreiraPosted almost 2 years ago
Hello EleneDzneladze! I hope you are doing well!
First of all congratulations to complete the challenge!
I have some suggestions that might be useful!
HTML
- You must put an attribute in the html tag (the first below !DOCTYPE html), the attribute of language like:
<html lang="en">- You must correct the first tag meta the attribute charset is incorrect ''uft-8" it should be:
<meta charset="UTF-8">- In the img tag, you must put an alternate text, the attribute "alt". The required alt attribute provides an alternate text for an image, if the user for some reason cannot view it (because of slow connection, an error in the src attribute, or if the user uses a screen reader). Source.
- You need to put whole code of the <div class="container" inside the tag <main> that indicate the card is a part of the main page, it's cause semantic. Here.
- The link immediately below your css link "index.css" is uncompleted you should remove it
- You can put in a folder that you should name "src" (source) other folders like 'image' and 'styles' to promote more organization in the project.
I hope you find it useful! Above all, the solution you submitted is great!
Happy coding!
Marked as helpful0
Please log in to post a comment
Log in with GitHub
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