Code Review, my project(landing page, Html,CSS,Js,gsap). "KillerWhale" Surfing school responsive website.

#html #scss #javascript #gsap

Hello Everyone 🖐
If You find some time for code review and evaluation of my project, I will be very pleased ...
Thank you in advance for every tip and attention. I accept any constructive criticism.😋 See you next time and
Have a fruitful and beautiful day !🌞

https://killerwhale.netlify.app/
https://github.com/MarioG8/Surfing-School-Full-Responsive-Website

Top comments (4)

menomanabdulla • Aug 20 '21

Hello, Mario. I could divide it in two segments, pros and cons.
First cons:

You can put more effort in responsive, I checked some problem arise with some device most importantly (760px-568px) viewport devices.
I believe you could follow DRY principle in your JS file though I don't know your JS skill weather you are beginner or just on other level. For this you can go with any design pattern. If you don't know what is design pattern is, please do some query on it.

Now pros:

You tried to use BEM naming convention, which looks very professional with your markup.
You use SCSS with it's super cool features like variable, mixin I appreciate it.
Animation is good seems to me.

All of points here from my perception don't take it personally please, stay healthy.
Happy hacking.

Mario Golik • Aug 21 '21

Thank you very much for your advice and feedback...😉 I'd like you to give a simple example of how to improve my JavaScript. There is not much code, so if you find 5 minutes, I will be very pleased .... Yours sincerely. Have a nice and fruitful day 😃
It is never late to learn..🐱‍👤💻

menomanabdulla • Aug 22 '21

"It is never late to learn.." I love you quote. Thanks for understanding. Before 5 minutes session could you please check my this repos js code?

Mario Golik • Aug 22 '21

I got what's going on..😃 Thanks for your precious advices !😉👍