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

Still using some different approach to minimize^

@pikapikamart

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


Feel free to drop your question or comments about my solutions^^

Community feedback

@thetechacks

Posted

Hey there it looks amazing !!

I just noticed that in the mobile view when the hamburger is clicked a green color outline is formed and if the links in the mobile menu are clicked the underline in the link looks odd...

I even noticed that when the menu is active the background is not a linear-gradient.

Just that, though overall it's amazing :)

1

@pikapikamart

Posted

@thetechacks Hey, thank you for seeing my solution :>>

About that green, I really did put that in there, it is a box-shadow when the hamburger menu gets focu.

About the line, oh yes I did not see that hahaha well the reason for not removing those is that, since they are a link right. The user won't be seeing those transition since they will be redirected in another supposed webpage or something like that, so removing those is not really necessary, if this were a real site where there are other pages.

For the last, keen eyes you got there hahahaha I'll implement that when I got the feeling but thank you really for that.

Thank you for reviewing^^

0

@thetechacks

Posted

@pikamart Oh the hamburger box-shadow looked as though it was a mistake :) Although I found another problem:

I think the mobile overlay div has a pointer-events: none value so that the hamburger menu could be closed, but it also causes few problems. I could select text and image even though the hamburger menu is open.

Hope it's helpful =)

1

@pikapikamart

Posted

@thetechacks Hey hahahaha you are really good on spotting those one and yes I did that for the button but like you said, the text can be selected and the button as well because the overlay is present, but it does not hide logically everything behind it.

Just to fix this, I will add again the pointer-events then just add z-index on the menu button.

Thank you really, though I won't implement this right away, I am teaching lots of my schoolmates about DSA in python and studying backend at the moment. But really, thank you so much^^

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