Hi
I hope this helps
- remove the width and height attributes from picture source and img. Even if you did need to include them, the purpose is ensuring the aspect ratio is correct and you would not include units in their values
- remove explicit width from the button. Just use padding
- you must remove the overflow hidden from page! This completely breaks the solution on mobile landscape
- 460px is too small to switch to the large screen layout. You should only change the layout when there is room to do so
- Also media query values should be in a responsive unit like rem
- you should only need one media query on this
- either use flex values (grow shrink basis) or max-width if using flex for the two halves of the layout on larger screens; OR us css grid
- keep gaps small. Don't try to create the whole layout with huge margins
I'm looking through code on a phone so it's hard to advise further tbh
Marked as helpful
0
