@BT453567
Posted
Hello
Excellent solution. There are many parts I like about this solution.
I have identified a small issue where the logo is not visible in the mobile menu.
If you uncheck the text indent the logo is visible in the menu.
Rather than re-creating the facebook and twitter logo for the mobile menu, you could manipulate the DOM from Javascript, by using the following commands (This is taken from my solution):
footerSocialsContainer.remove(); mobileMenuFooterContainer.appendChild(footerSocialsContainer);
I like how you calculated the distance of the graphic to edge of the screen and also how you implemented the tab buttons.
I also found it took longer than expected to code.
Hope that helps
Marked as helpful
@Yejin-Han
Posted
@BT453567 Thank you for your feedback! I miss that logo visibility issue. I fix it. Thanks :) I didn't manipulate the DOM because I thought it would be bad in terms of performance or maintenance. But I'll try it next time.