Huddle Landing Page || HTML CSS Grid

Community feedback

• 

  Hussein Samy•340

  @HusseinSamyPosted 11 months ago

  Hello @ofthewildfire, 👋🏻

  Awesome job on taking the first steps towards your dream!

  I'm thrilled to see your engagement with this platform to enhance your design skills. However, I have some suggestions for you:

  • Adding meta tags to enhance the SEO, such as:

  <meta name="title" content="My Website">
  <meta name="description" content="My Website Description">
  <meta name="keywords" content="Some, Keywords, Here">
  
  

  • Correct me if I am wrong, but I think that you are using the modifiers (in BEM naming convention) at the wrong place – they should be used to changes the appearance of the block. Here is their docs

  • Leaving some space and writing comments in your HTML can save you sometime when navigating your code, something like this:

  
  < ========== >
  < Left Section >
  < ========== >
  <section>
  
  <-- Left Image -->
  <img>
  
  <-- Attribution --> 
  <div>
  </div>
  
  </section>
  
  
  < =========== >
  < Right Section >
  < =========== >
  <section>
  ...
  </section>
  

  Hope you find it useful!😇

  Keep up the fantastic work, and happy coding! 🚀

  Marked as helpful

  0

  ---

  

  Kirsten ✨•460

  @ofthewildfirePosted 11 months ago

  @HusseinSamy Thank you for the suggestions and corrections :) I wasn't using BEM at all 😂but you've reminded me I do have to learn it, so I'll learn the BEM method and implement it ASAP.

  I'll also add comments ✨♥️

  1

  

  Hussein Samy•340

  @HusseinSamyPosted 11 months ago

  @ofthewildfire Actually, you're using it in your own way 😂, it's easy don't worry. Wish you all the luck ✨

  1