@Yejin-Han
Posted
First of all, thank you for your hard work. It must have taken you quite a while to re-layout the mid-size screen.
I'll briefly tell you the problem I've checked. There is an error that the mobile version of the menu is maintained if you open the menu once in the mobile size and resize it to the desktop size. There are parts that are very slightly different in layout or size from the given design, but overall, it seems to have been completed well.
In addition, it's amazing that you scripted the slide yourself! However, there are many parts that seem to have been scripted even if you don't have to use the script, so I'm worried that if there are more of these codes, there will be performance problems. It would be good to consider creating a class and toggle it.
Thank you.
Marked as helpful
@tortaruga
Posted
Thank you @Yejin-Han!
I hadn't noticed the issue with the mobile menu on resizing the screen, I fixed it now.
And yes you're right I think I made it all more complicated than it needed to be 😅 I'm not very good at this yet, so sometimes my solutions are not very efficient. I'll keep it in mind in the future. Thank you again for your comment, I appreciate it very much!
