Hi Prathna, **First of all, congrats on finishing the project!** Here is my feedback to you, I do agree with IryDev in the following: - Try to use Semantic HTML, this is mainly used for two reasons, accessibility, and readability, in my opinion using a

should be the last thing you should consider using, there are other tags with more meaning such as

, and so many more, please check this article to get familiarized with Semantics in HTML. - Divs are ok to define your content inside the container, but never ok to use them as the container itself, For example: ``` **This is not right**

Hello World

Some example text here>

**This is correct**

Hello World!

Some example text here

``` - Avoid using divs for heading text, this text should've been an H1 instead: **Improve your front-end skills by building projects**. - I'm not sure why your image looks pixelated, don't give hardcoded dimensions e.g. you added a width and height of 330px, a better approach is putting your image on its own container and putting a width of 100% of that parent container, it would make your image become responsive regardless of the viewport size (screen size). I leave you these resources for you to improve your coding skills even more: - [Semantic HTML](https://web.dev/learn/html/semantic-html/) - [Specificity](https://web.dev/learn/css/specificity/) - [Images Best Practices](https://www.volusion.com/blog/diy-responsive-design-part-iii-css-and-image-best-practices/#:~:text=Typically%2C%20it's%20good%20practice%20to,to%20respond%20to%20available%20space). A good course to deeply understand HTML and CSS: - [Understanding HTML and CSS](https://www.udemy.com/course/understanding-html-and-css/). **Hoping this helps you improve**

Manuel Matus

All solutions

Responsive Newsletter Sign up form

QR Code Component