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

Responsive landing page (Mobile-First approach). Using ReactJS, SCSS.

#react#sass/scss#bem
Simon Hernandezβ€’ 615

@simonhernandez

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


Hi Guys!

It's been a while since I last submitted a solution. I am very happy to be back!

This time around I decided to put into practice my React skills to bring to life this simple and beautiful design. Used Semantic HTML tags, SCSS for styling, and AOS library for animations.

Hope you guys like it! I also would really appreciate any feedback on my code or on the overall project.

Wish you guys an awesome week and fun coding!

Community feedback

P
ApplePieGiraffeβ€’ 30,545

@ApplePieGiraffe

Posted

Greetings, Simon Hernandez! πŸ‘‹

Incredible work on this challenge! πŸ™Œ It's great to see you back with another solution! πŸ‘ I really like the animations you added to your solution! πŸ˜€

A small suggestion I have is to perhaps set the alt text for the icons in the "Why choose Easybank?" section to be an empty string so that they will be ignored by screen readers. That’s because those elements aren't really necessary or important to the content of the page and as a result probably don't need to be read by screen readers. πŸ˜‰

It would also be good to add some focus styles to some of the interactive elements on the page that do not currently have them (such as the "Request invite" buttons).

And it would be worth taking a look at the responsive issue that @veljkoilic mentioned in the hero section of the site! πŸ™‚

Hope you find this helpful. 😊

Keep coding (and happy coding, too)! 😁

Marked as helpful

2

Simon Hernandezβ€’ 615

@simonhernandez

Posted

Hello, @ApplePieGiraffe!

It is great to know about you again!

Thanks so much for your feedback and suggestions. I took them into account and applied them on my solution. I also fixed the responsive issue @veljkoilic mentioned.

Thanks again for your always helpful feedback!

1
P
ApplePieGiraffeβ€’ 30,545

@ApplePieGiraffe

Posted

@simonhernandez

Of course! Happy to help! 😊

0
Veljko Ilicβ€’ 190

@veljkoilic

Posted

Awesome work dude!

Everything looks super slick and responsive! I suggest you make your animations go only one way. If you scroll up and down again they replay, but this is just a personal taste.

However, there is a bit of a weird 1000px - 1140px area. The Intro section heading overlaps with the background making it unreadable and the articles at the bottom are really squished. This is all minor but I thought you'd appreciate the nitpicking feedback!

Marked as helpful

1

Simon Hernandezβ€’ 615

@simonhernandez

Posted

@veljkoilic

Hello Veljko!

Thank you so much for your feedback!

I don't know how I missed such an obvious issue in responsiveness. I took a look at it and fixed it so that the heading and background image don't overlap. Also fixed the squished articles by setting some gap between them.

Thanks again for your helpful feedback on my solution. I really appreciate it!

0

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