Skip to content
Submitted almost 3 years ago

CSS Grid, CSS Flexbox

Adedayo•200
@pleasantvik
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Easybank landing page
  • HTML
  • CSS
  • JS
3intermediate

Design comparison

SolutionDesign

Solution retrospective

This is my first challenge, please help review the code and point out neccesary correction to be made. thanks

Community feedback

  • Aakash Verma•6,710
    @skyv26Posted almost 3 years ago

    As I can see my friend, you have used font-size bigger than required, which change your design look and feel totally different from original one.

    I also checked it on my mobile phone. I think you have not made it fully responsive.

    I saw that somewhere your text and images both are not aligned to center.

    Overall, I would like to say that it was your first challenge that you have completed and I am happy that you prefer quality work. So fix your issue that I wrote above.

    I hope you understand and best of luck for your future 👁️

    🤞

    2
  • Adedayo•200
    @pleasantvikPosted almost 3 years ago

    Thanks for the feedback, really appreciate… I will work towards improving it

    1

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord