Hi, I can spot some problems here so hope these pointers are helpful :
- The image is important content so should have a proper alt description.
- Personally I'd consider the preparation time text as a heading.
- The table must include header cells for the table heading content. Ideally these should include a scope attribute set to row.
- Just so you're aware, you don't need to include paragraphs inside list items.
- Don't use
b
tags. That's a deprecated tag. Use strong instead for strong emphasis. - You don't need to repeat the preconnect google font links. Only ever include them once, no duplication.
- Optionally you can load all the fonts in one go, choosing the specific families and weights you need. This would be better for site performance.
- Get into the habit of including a full modern css reset at the start of the styles in every project. Andy Bell or Josh Comeau both have good ones you can look up and use.
- Set the max width in rem not px. This is important for users who change their default text size as it means the layout will scale correctly.
- The max width on the image must be 100% and not a pixel value. I think that is what is breaking this site when I view it on mobile. Its making the content overflow on both sides (I'll add a picture to discord so you can see what's broken).
- The other thing that is likely causing the overflow is the explicit width on the name class. Those table cells should each be width 50% to make them even.
- This challenge does not need any media query. You can remove them and all should work once you've applied the above fixes. But in future challenges make sure you work mobile first, with the larger screen overrides in the media query. Have a read of this: https://fedmentor.dev/posts/responsive-meaning/