Chevalier Antoine• 670
@SeyBoo
Posted
Hello great solution i love the animation that you provide it's simple but add a lot to the website.
I suggest you some point that you can improve :
Some visual improvement :
- Maybe add a
max-width
on the main element because on larger screen we see 2 row , one with 5 box and one with 3 box. - Add a
height: 100vh;
on the container element so that i can take the full height of the screen.
JavaScript improvement:
- A easier way to do the toggle is just to add the class 'light' in the body element.
For the SCSS part your code is very well cut and organised and seem well typed so well done.
A overall great solution keep going !
1