Skip to content
Submitted about 5 years ago

HTML, CSS, Flexbox

Lukasz•315
@devluke88
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Base Apparel coming soon page
  • HTML
  • CSS
  • JS
1newbie

Design comparison

SolutionDesign

Solution retrospective

There is no JS and Sass (learning in progress);

Community feedback

  • Matt Studdert•13,611
    @mattstuddertPosted about 5 years ago

    Nice work Lukasz! Good to see the foundations are in place for when you build your JS skills. When you add an input element, always be sure that it either has a corresponding label or an aria-label attribute so that screen reader users know what the field is for.

    Keep up the great work! 👍

    0
    Matt Studdert•13,611
    @mattstuddertPosted about 5 years ago

    @devluke88 you're welcome! I've just checked your update. The label needs text inside to work for screen reader users. You can then use a class to hide the element visually so that it's available only to screen reader users.

    Let me know if you have any questions.

    0
    Lukasz•315
    @devluke88Posted about 5 years ago

    @mattstuddert Thank you for your help.

    0
    Lukasz•315
    @devluke88Posted about 5 years ago

    Updated;) Thank you for your feedback!

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord