I think this solution looks great (viewing on mobile)
Only small suggestions really
- make sure content doesn’t hit the top of the screen on mobile
- on your next one, work mobile first. Code is usually shorter and is more performant that way
- class names can be whatever you like, but it looks like you’re sort of attempting BEM naming on this? If so, it’s not quite right as it would be
Block__Element Block__Element—modifier. Read a bit more about it if that’s what you’re trying to do
All good stuff on this though, well done
Marked as helpful
0
