@abiodunolunuSubmitted almost 4 years ago
Hi Everyone, I would entertain feedbacks, constructive criticism and questions. Thank you.
Jadson Santos
@santos-jadsonAll comments
- @santos-jadsonPosted almost 4 years ago
Hello, Olunu! How u doing?
Your code is great and looks very similar to the original design. Congrats :D ... I think the only thing you could check is the fact that some of the cards don't have the same height. Maybe if you set a min-height or a max-height property, it could be fixed.
2 - @ashwin366Submitted almost 4 years ago
Any feedback would be appreciated. Thank you in advance.
@santos-jadsonPosted almost 4 years agoHello, Ashwin! How u doing?
Your solution is great and very similar to the original one. I have nothing bad to talk about it. I mean, the only thing you could consider is to create some hover effect inside your "Learn More" button.
Congrats :D
1 - @Mamun-AliSubmitted about 4 years ago@santos-jadsonPosted about 4 years ago
Hello, Mamun-Ali! How you doing?
Your project is great and very similar to the original one, but there's two things you should consider reviewing later. The gap attribute only appears in big screens. Also, in some resolutions the layout is breaking, there's divs staying behind other divs. But don't worry, that's not a huge issue.
Happy coding!
Marked as helpful2 - @AndreaLecuonaSubmitted about 4 years ago@santos-jadsonPosted about 4 years ago
Hello Andrea, how you doing?
Your code is really good and very similar to the original design. I don't have anything bad to say, I mean, the only thing I think you could change is the responsiveness, because when I open the site in a resolution bigger than 375px, the layout is breaking. But don't worry, your code is great and that's not a huge issue :D.
1
