Mobile First Workit Landing Page using HTML & SCSS

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 1 year ago

  Hey, Syd! 👋

  Congratulations on completing your first Frontend Mentor challenge! 🎉 Nice job on this one! 👍

  A few things you can do to improve your HTML would be,

  • Turning the features section into a list using the ul and li tags (since that is a list of features).
  • I don't think the word "tailored" in the page heading needs to be a link, as it probably only has that underline for decoration.
  • Not adding any alt text to images you don't want screen readers to notice and read (such as decorative images, which aren't important to the content of the page).
  • For the above reason, it might be worth removing the alt text on the social media icons in the footer of the page and adding an aria-label to the links that wrap them instead.

  I don't think using Sass for this project was overkill. Sass has lots of great features that are easy to get used to and using it here is great practice for larger projects where it comes in very handy. 😉

  Making layouts responsive and using absolute positioning are two things that become much easier to do (and think about in general) with practice! 😉 Something that really helped me become better at the two is inspecting other websites and reading other people's code to see how they did things. Use the dev tools to inspect and poke around professional websites or browse this platform and the read the code of high quality solutions—this will show you ways of doing things that you might not have thought of before and give you a chance to explore new ideas. 🙂

  Hope this helps. 😊

  Keep coding (and happy coding, too)! 😁

  Marked as helpful

  1

  ---

  

  P

  Syd•20

  @SydBrainPosted over 1 year ago

  @ApplePieGiraffe

  Thank you very much! Your feedback was extremely helpful!

  1