dark theme landing page

Community feedback

• 

  Fexxix•230

  @FexxixPosted over 1 year ago

  It looks professional and the resposiveness is also good. I won't point out the alt img text problem since @Finney06 already briefed you on that. The only thing I noticed that was off was the input at the bottom of the page. It could use a bigger font and a placeholder text. You can get rid of the default border and outline by:

     border: none; 
     outline: none;
     outline-color: transparent; // this is preferable if you are taking contrast theme users into account
  

  Then you could utilize box shadow or a border to match your theme and give it a pleasing look. Other than that, superb stuff. Happy Coding!

  Marked as helpful

  0
• 

  Finney•3,030

  @Finney06Posted over 1 year ago

  Hello there 👋. Good job on completing the challenge !

  Here are some suggestions regarding your code that may be of interest to you.

  HTML 🏷️:

  To clear the Accessibility report:

  • Images should have alternate text. <img src="images/icon-memory.svg" alt="memory icon"> this provides alternative text descriptions of images for people who are visually impaired and rely on screen readers to navigate the web. Screen readers can read the alt text aloud, enabling these users to understand what the image represents and how it relates to the content on the page. Overall, including alt text in HTML is an important best practice for creating accessible and user-friendly web content.

  • Always avoid skipping heading levels; Starting with <h1> and working your way down the heading levels (<h2>, <h3>, etc.) helps ensure that your document has a clear and consistent hierarchy.

  I hope you find it helpful!😏 Above all, the solution you submitted is 👌. 🎉Happy coding!

  Marked as helpful

  0