Skip to content
Submitted about 2 years ago

job-listings-with-filtering

Adeniyi Smart•520
@Iamweird2
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Job listings with filtering
  • HTML
  • CSS
  • JS
3intermediate

Design comparison

SolutionDesign

Solution retrospective

Feedbacks highly appreciated :)

Community feedback

  • Adriano•38,270
    @AdrianoEscarabotePosted about 2 years ago

    Hi Adeniyi Smart, how are you?

    I really liked the result of your project, but I have some tips that I think you will enjoy:

    • <html> element must have a lang attribute
      • To improve the accessibility of the project you could have put an h1. Every page must contain a level 1 header, for people who use screen readers, identity what the main title is and follow the sequence h1-h5

    The rest is great!

    I hope it helps... 👍

    Marked as helpful
    0
    Adeniyi Smart•520
    @Iamweird2Posted about 2 years ago

    @AdrianoEscarabote thanks man, I'll make the corrections

    1
  • Avinash DV•150
    @avinashdvPosted about 2 years ago

    Hi, The webpage looks really cool, nice work. A minor UI bug I came across: On selecting the tags, if there are only 1 or 2 jobs, the background color is covered for only the 2 jobs instead of entire body. Changing the h-auto to height:100vh on the div which is inside <div id="__next"> will solve this issue. I hope this helps, happy coding.

    Marked as helpful
    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