Looking to hire developers?
check my animated hamburger
@Mahesh-yadav
Posted
Hi Great solution. Your cross icon is not perfectly centered. I also faced this problem. My solution is that while applying transform change the order - first translate both lines so that they lie above each other and then you can rotate.
@ovidiuantonio
Posted
Hello, very nice solution! It's very responsive, looks very nice and the hamburger is nice done! You did a great job! One thing for the burger that you can add is cursor: pointer; to the entire hamburger div, not for the lines that form it because if you hover the cursor between the lines the pointer disappears.
Happy coding! Keep going!
@Da7y
Posted
@ovidiuantonio thx for the advice
@Da7y
Posted
@ovidiuantonio why do i need cursor: pointer; in mobile size? will it make a different?
@ovidiuantonio
Posted
@Da7y cursor: pointer; on clickable elements helps you create a better user experience (ux)
@Da7y
Posted
@ovidiuantonio ya but in mobile there is no cursor ,so i think it's an useless line of code and should have removed it. if u have other opinion plz tell me
@mattstuddert
Posted
@Da7y Iād still recommend adding it. People with poor eyesight will often zoom in their screen. This will trigger the mobile layout. So you could still have people who are using a mouse on the mobile layout on your site.
Hope this helps!
@Da7y
Posted
@mattstuddert oh i didn't knew that thx!
@mattstuddert
Posted
@Da7y you're welcome! š
@Da7y
Posted
great idea thx man
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
