@RaidEyes
Posted
There are many problems with your code:
-
You are using the desktop approach first which is not recommended, usually if we have the design for both desktop and mobile, we usually go with mobile first. You should only go for desktop first if the desktop design is all you have.
-
Regarding your
<header>
, it seems that you didn't apply any style to the<ul>
element. From what I see you change your components completely, this is not correct. You should apply the style to themenu
when the screen size changes. -
Your
React
components render every time something is triggered which is not good for performance. Instead, when you click something only the related content re-render, this is also the point ofReact
, using components to reduce complexity and increase performance.
Overall, still good work! Happy coding! Hope you find this helpful!