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

Community feedback

• 

  Godwin Martins•180

  @gmcodes20Posted about 3 years ago

  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

  2

  ---

  

  Danny•200

  @DannyLenkPosted about 3 years ago

  Hi, @gmcodes20,

  Thank you for taking the time and review my work. Much appreciate! Your comments definitely helped me find mistakes and fix them. Now the website looks more consistent. I would be glad to hear from you further down the road.

  1

  

  Godwin Martins•180

  @gmcodes20Posted about 3 years ago

  @DannyLenk I'm glad I was able to help. I definitely will love to talk to you again

  0
• 

  darryncodes•6,370

  @darryncodesPosted about 3 years ago

  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

  1

  ---

  

  Danny•200

  @DannyLenkPosted about 3 years ago

  Hi, Darryncodes,

  Now it works. I never would've guessed it, probably. Big thank you!

  Have a good one.

  0