Skip to content
Submitted over 5 years ago

Stack Overflow, W3Schools

Axel•150
@Axel01011111
A solution to the Project tracking intro component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

There, I fixed the mobile design. I like how it looks now. Still, I'm sure there are things that I should keep fixing

As always, feedback will be appreciated.

Code

Please log in to post a comment

Log in with GitHub

Community feedback

  • Matt Studdert•13,611
    @mattstuddert

    Nice work, Axel! Your desktop layout looks really good. Here are a few pointers after looking at your code:

    • You've got <div class="Header"> inside a <section>. Instead, I'd recommend removing the section and changing the div to a header and then remove the class. Semantically it makes more sense.
    • Avoid uppercasing text in your HTML. This is because some screen readers will read these words out letter-by-letter, which makes the content inaccessible. Instead, you can use text-transform: uppercase; in your CSS to get the desired effect and keep the content accessible.
    • You've got IDs on your HTML elements, but I couldn't find anywhere that you're actually using them. So you might as well remove them from your HTML. Unless I'm missing something!

    I hope that helps. Keep up the great work!

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