Skip to content
Submitted over 3 years ago

CSS Flexbox

Oluwakemi Omoyeni•300
@Captressketh001
A solution to the Digital bank landing page challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

I will love to have a feedback on how I can make the toggle better

Code

Please log in to post a comment

Log in

Community feedback

  • Rémi Martineau•355
    @MartineauRemi

    Hey ! You did a great job :) Concerning the menu toggle, I think it should work with only this in your main.js file :

    $(function() {
    $(".toggler").on("click", function(){
        $(".nav-items").toggleClass("active");
    })
})
  • Sam Guliker•540
    @Sam-Guliker

    Hi Captress,

    You're almost there. You can still improve the HTML by using aria labels on the <a> tag. Next to that check the headings, H5 doesn't seem neccisary.

    Design wise, check the fonts, paddings and buttons.

    Have a good day

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