Hey. Nice job making the site responsive, you have done well! Here are some things I want to share with you: 1. **Avoid** the inline styling in your .html files as much as possible. Sure, you may apply some inline styling in your code, but styling the whole page in the index.html file is considered a bad practice 2. I know you asked for some advices for making a responsive page, but here is also a really important thing - **HTML Accessibility**. I am not really good at it either, but here are some main takeaways related to your code: - Wrap the main content of your page in the `

` tag, which is your product-preview-container div and the attribution div - It is not a very important thing here really, but wrapping the content of the product-preview-container div in an `

` tag would be better in terms of accessibility. Remember this for your future projects. [Here](https://www.w3schools.com/tags/tag_article.asp) is some information about this tag. - **Headings**. First of all, any page should have at least one `

` heading and that would be, let me put it like this, - the main heading that describes your whole page. Here it would be the name of the perfume - Gabrielle Essence Eau De Parfum. Another thing - headings on your website should follow the order - `

`s, `

`s etc. should not come before the

heading. Obviously, that rule works for other tags as well. Just remember this: the `` tag per page), all of your `
`s come next, all of your `
`s come next and so on. Since we made the name of the perfume our `
` heading, we will not make the "perfume" text above it an `
` heading, since that would break the order of headings on the page, therefore I would recommend making it a paragraph(`
`). - The attribution div. It would be a better and a more semantic approach, in my opinion, if you made it as the `
`. And, if I remember correctly, the `` tag. - It would be great to ensure accessibility for screen readers when displaying a price with a discount and the original price, but I have not come up with the solution for this yet, unfortunately. 3. It will be a bit better in terms of readability and maintenance for your code if you keep all of your CSS resets on top of it and then start writing the general styling. The way I tend to do it - custom properties come first, then the reset and then the styling. 4. The way I use `@media`s is I write them right under the classes that I will be applying those `@media`s for. Maybe that is not the case for you, but for me it is a lot easier to maintain the classes used in my `@media`s when they are next to the original classes. Good luck with your web development journey! :)

` heading and that would be, let me put it like this, - the main heading that describes your whole page. Here it would be the name of the perfume - **Gabrielle Essence Eau De Parfum**. Another thing - headings on your website should follow the order - `

`s, `

`s etc. should **not** come before the

`s come next, all of your `

`s come next and so on. Since we made the name of the perfume our `

` heading, we **will not** make the "perfume" text above it an `

` heading and that would be, let me put it like this, - the main heading that describes your whole page. Here it would be the name of the perfume - Gabrielle Essence Eau De Parfum. Another thing - headings on your website should follow the order - `

`s etc. should not come before the

` heading, we will not make the "perfume" text above it an `

gabbyhunter1

All solutions

Recipe page challenge HTML & CSS

Product preview card component challenge