Hi
This looks good on my mobile.
Although I can see some room for improvements in places...
- it's not good practice to clutter the html with empty divs. If you want to include decorative imagery, either put them in the html or use pseudo elements.
- those buttons should all be anchor tags not button elements (they would trigger navigation)
- you're making the styles way harder than they need to be by setting min and max width and setting padding in %. If you put padding in rem it's consistent and controlled. And all these need are max width, no min required
- similarly, there's no need to set max width on low level elements like paragraphs. Let them bee as wide as they need to - they are already limited by the max width and padding on their container
- always remember to include focus visible styles (obvious and clear ones, not included in designs) as well as hover styles on all interactive elements
I hope this is helpful
Marked as helpful