optimusprime202• 1,200
@optimusprime202
Posted
Hey @Ishan-Sanjaya, Exactly right!
Marked as helpful
1
Luke Fernando• 130
@Luke-Fernando
Posted
@optimusprime202 Thanks man really appreciate.
1
Design comparison
Solution retrospective
Any feedbacks are welcome.
Community feedback
Travolgi 🍕• 31,420
@denielden
Posted
Hi Ishan, I took some time to look at your solution and you did a great job!
Also I have some tips for improving your code:
- in the mobile background add
background-repeat: no-repeat;for the image bg - I see that in the mobile menu has some problems:
firefox does not support the css
backdrop-filterproperty, you can add an exception in firefox by putting a solid color there only through@ supports
Look here:
Overall you did well :)
Hope this help and happy coding!
Marked as helpful
1
Luke Fernando• 130
@Luke-Fernando
Posted
@denielden Oh! Thanks, Daniel, I will definitely update my code for the firefox browser.
1
Travolgi 🍕• 31,420
@denielden
Posted
@Ishan-Sanjaya You are welcome! I would really appreciate if you mark my comment as helpful if it helped you, thank you very much :)
1
Please log in to post a comment
Log in with GitHub
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