Responsive landing page (Mobile-First approach). Using ReactJS, SCSS.
Design comparison
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
- @ApplePieGiraffePosted over 2 years ago
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 helpful2@simonhernandezPosted over 2 years agoHello, @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@ApplePieGiraffePosted over 2 years ago@simonhernandez
Of course! Happy to help! π
0 - @veljkoilicPosted over 2 years ago
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 helpful1@simonhernandezPosted over 2 years ago@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 GitHubJoin 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