meet-landing-page-sass

Community feedback

• 

  P

  Tuna Erten•360

  @tunaertenPosted 3 months ago

  Hello there 👏 You've done a pretty good job.

  Your project works responsively and aligns well with the design. I have a few suggestions:

  • You can use the <picture> tag with min-width to display different images for various screen sizes.

           <picture>
            <source
              media="(min-width: (your value))"
              srcset="./assets/desktop/image-hero-left.png"
            />
            <img
              src="./assets/tablet/image-hero.png"
              alt=
            />
          </picture>
  
          <picture>
            <source
              media="(min-width: (your value))"
              srcset="./assets/desktop/image-hero-right.png"
            />
            <img
              src="./assets/tablet/image-hero.png"
              alt=
            />
          </picture>
  

  • Also, don't forget to use cursor: pointer for all buttons.

  I hope my suggestions have been helpful. Happy coding!

  0