Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Easybank landing page using css

Ponnusamy 250

@ponnusamy-krish

Desktop design screenshot for the Easybank landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
3intermediate
View challenge

Design comparison


SolutionDesign

Solution retrospective


Share your thoughts....

Community feedback

@simonhernandez

Posted

Hello, Ponnuasamy! 👋

Great work on this solution! I really like that it is responsive, and the fact you added a cool transition on the mobile menu.

Looking into your code I thought of some suggestions you might find helpful:

  • On the Articles Section I noticed you gave a class to each of the individual articles (articles1, articles2, and so on) with the same styles. I think it would be more efficient to create a single .article class containing those styles and applying it to each article element, that way you get rid of redundant CSS. Same with the .navtext classes.

  • I would consider setting a max-width on the Article elements, so that they do not stretch all the way on tablet screen size. Same with the Feature elements.

Hope you find this feedback helpful!

Marked as helpful

1

Please log in to post a comment

Log in with GitHub
Discord logo

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