HTML, CSS, JavaScript

Please log in to post a comment
Log in with GitHubCommunity feedback
- @EDrick-her
There are too many <div> elements in the HTML. Try to use semantic tags to make it more readable, like <article>, <footer>, and <section>. The design is not centered and it's not responsive. I think you used position: relative for everything, but you could have used flex or grid. I would recommend using grid, but that's just because I find it easier, especially for handling @media (media queries). I would recommend using an extension or tool to help you get the exact colors you're working with so you can match them properly. I liked the button animation, but it only happens on the first click. After that, it doesn't appear again, and when you click it, it creates a type of horizontal scroll that shouldn't be visible. I loved your GitHub; I think you manage it well, unlike mine, which is a mess haha. That's all. If you need anything I can help with, just let me know. Take care!
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