Four-Cards

Community feedback

• 

  Hugo Sanchez•270

  @HunigoleSanPosted 10 months ago

  Hello! 👋 @AlyssonDemari

  Congratulations on successfully completing the challenge! 🎉 However, let me provide you with some tips to help you improve as a web developer. Take some time to reflect on these suggestions, as I'm here to assist you in your improvement journey. 💼

  First tip: Width in the body tag 📏

  In your code snippet, you've set a fixed width for the body tag using "width: 1440px". While this isn't necessarily wrong, it could lead to unnecessary scrolling. Additionally, you're using "margin: auto", which becomes redundant when you're already utilizing flexbox for centering elements.

  Furthermore, within your media queries, you're setting "width: 100vw", which can also cause unnecessary scrolling. Instead, consider removing these width declarations as your "section" tag already utilizes CSS GRID, allowing child elements to dynamically provide their width.

  Second tip: Width in the Section tag 🖥️

  Similarly, if your child elements are dynamically expanding within the section tag, there's no need to specify a width for it. Removing these width declarations will make your code more adaptable for various devices.

  Remember, your effort and current knowledge are appreciated by developers, but there's always room for improvement. Keep practicing and striving for excellence on your journey as a web developer. 🌟

  1