GSAP Practice and TailwindCSS with GuLP

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, DPSumner! 👋

  Good effort on this challenge! Your solution responds well. I really like animations that you added. Here's my few suggestions:

  • Since you're using a lot of animations, read about prefers-reduced-motion CSS media feature which is used to detect if the user has requested that the system minimize the amount of non-essential motion it uses.
  • It'd be a good idea to make logo clickable, so wrap your logo into <a> anchor tag and then change alt text to more descriptive like "Meet - home page". If you don't want that, since your logo image is decorative your alt text should be provided empty (alt="") so that they can be ignored by assistive technologies, such as screen readers.
  • "Smarter meetings, all in one place" paragraph should be a heading <h1-h6>.

  Good luck with that, have fun coding! 💪

  1
• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hi, DPSumner! 👋

  I know I'm a little late to this submission, but ooh—I just saw this and realized we both used GSAP and ScrollTrigger for the same challenge! 😆 I like the animations that you added and I think you did rather well on this challenge! 👍

  This is a super tiny issue, but the avatar images to the right side of the screen seem to be overflowing the body in the mobile layout. You might want to fix that so that there isn't any extra white space to the right side of the layout of the page. 😉

  Keep coding (and happy coding, too)! 😁

  0