HTML, CSS Flexbox

Community feedback

• 

  Bhuvnesh Upadhayay•2,000

  @bhuvi819381Posted 2 months ago

  Hello Brother,

  I checked your solution, and it’s great! Here are a few suggestions to improve it further:

  1. 📄 Update the README file
     Use the provided README template, update it, and upload it for better clarity and professionalism.

  2. 🏷️ Wrap everything in the <main> tag
     This ensures proper structure and accessibility.

  3. 🖼️ Don’t leave the alt attribute blank
     Except for design-related images, always provide a meaningful alt text for accessibility and SEO purposes.

  4. 🔠 Limit the use of <h1> tags
     Use a single <h1> per page, followed by <h2>, <h3>, and so on, to maintain proper hierarchy.

  5. 🚫 Avoid using px
     Opt for relative units like rem or em for better responsiveness and scalability.

  6. ✂️ Use margin-inline and margin-block
     Instead of separate margin-left and margin-right, use margin-inline. Similarly, for margin-top and margin-bottom, use margin-block. It simplifies your CSS and looks clean. 😁
     Using this looks cool 😄 So just telling. Don't mind it

  These updates will make your solution even better! Keep up the fantastic work! 👏

  0