CSS custom properties, Flexbox, CSS Grid, Mobile-first, native JS

Solution retrospective
Yes, please. There is a thing that bothers me the most. Why could a hidden attribute not work in some places? For example, on the "destination" and "crew" pages' <article>s it works, whereas on the "technology" one it doesn't. The same problem was with the <div>s containing imgs on the technology page. So, I had to add some changes to the js files.
Please log in to post a comment
Log in with GitHubCommunity feedback
- @gmcodes20
Hi @danny, nice one you got here. Maybe you can change the mobile menu hamburger to a pointer. Also, the crew image should be at the bottom on the larger screen and if you can also remove the border-bottom from the image on the large screen
Marked as helpful - @darryncodes
Hi Danny,
I had this exact same problem and Grace came to my rescue, something I now won't forget:
Here's the info:
- the display property overrides the hidden attribute and you have in your code:
img, picture { max-width: 100%; display: block; }
- try adding:
[hidden] {display: none !important;}
All the best
Marked as helpful - the display property overrides the hidden attribute and you have in your code:
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord