flexible landing page with Tilting cards effect.

Community feedback

• 

  Tereza•605

  @sirriahPosted over 3 years ago

  Hello, nice work :). First, look at the report - there are some issues: More than one main tag etc.

  • I see that images are stretch to the content, but it is not necessary - look at object-fit: cover.
  • Please, move the css part into the separate file. Make code mobile first
  • It is not necessary to write text with capslock, use css text-transform
  • Dont use % for everything - see this article
  • the animation script vanilla tilt causes error in console. Look into developer tools in Chrome or in Firefox. The animation is little bit weird.
  • You use backslash in file path, but it should be / not \

  Happy coding!

  1

  ---

  

  Rahul kumar Gautam•655

  @RahulKumarGautam1636Posted over 3 years ago

  @sirriah it's first time, i got any suggestions and surely i'm going to solve the issues suggested by you. Thank you for your time..

  0

  

  Tereza•605

  @sirriahPosted over 3 years ago

  @RahulKumarGautam1636 Yes, I saw it, that you dont have any comments. Maybe that would be better, if you write first comment longer - try to write some questions what you want to know or what suprised you when you write the code. :) Good luck for next challenge.

  0