@Akshay-TardeSubmitted almost 2 years ago
It was nice to do DOM manipulation using JS. This challenge tested my planning of the layout. I tried to abide by the DRY principle while writing the CSS for this page.
Any feedback would be welcome!
Katleho-codes
@Katleho-codesAll comments
- @sankalp414Submitted almost 2 years ago
This was an amazing challenge to do, and I try to improve my skills. I would like to get some feedback from you and advice on what to improve.
@Katleho-codesPosted almost 2 years agoThis is incredible! Totally outstanding.
Marked as helpful0 - @alcias4Submitted almost 2 years ago
easy to practice api
@Katleho-codesPosted almost 2 years agoHey! Nice work, things to consider :
- The title of your submission has a spelling error (Fecht instead of Fetch)
- The font you used is not the one that's part of the project or is it the font weight
0 - @Katleho-codesSubmitted about 2 years ago
Project done. Please feel free to give feedback and constructive critisism
- @NadiaFrShLmSubmitted about 2 years ago
That was challenging for me.
Still not sure is it useful to split all the components to several files (typography, layout, breakpoints, variables...)
Sure the JS code could be more compact, but for now I'm proud of my result.
- @SrHatcherSubmitted about 2 years ago
Any suggestion is very well welcome :)
@Katleho-codesPosted about 2 years agoThis is great stuff. Great use of the CSS GRID
1 - @fisttySubmitted about 2 years ago
- Images with scroll in animation
- Also added overlay animation on the images
- Width animation on hover
@Katleho-codesPosted about 2 years agoThat's still cool as well. Great job though!
0 - @fisttySubmitted about 2 years ago
- Images with scroll in animation
- Also added overlay animation on the images
- Width animation on hover
@Katleho-codesPosted about 2 years agoHi! This is great stuff. Things to consider though:
- The footer on desktop screens has white social media icons
1 - @niyicodesSubmitted about 2 years ago@Katleho-codesPosted about 2 years ago
Hey! Nice work. Although the testimonial cards are nowhere the same as the original design. You could try using position: absolute, and also increase the padding in the navigation bar.
0 - @Andy-CODSubmitted about 2 years ago
Another completed project in which I have accustomed myself with the css transform property. Any comments on how I could improve or a different way I could have done this are welcome, thank you 🙏.
@Katleho-codesPosted about 2 years agoHi! This is so clean. I especially love the animation of the navbar on mobile when opening and closing. The responsiveness is on point.
1
