Skip to content
Submitted over 3 years ago

sass and js

Chrisā€¢740
@chrisbailey85
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

Any feedback will be greatly appreciated, thank you.

Community feedback

  • Emmilie Estabilloā€¢5,540
    @emestabilloPosted over 3 years ago

    Hi @chrisbailey85, looks good and responsive! I would add a max-width to the main content though, so that it doesn't look too stretched on larger screens.

    I think the filtering should also be tweaked. For example, if I click Javascript+Junior, the job posting for Faceit company is included in the list, which doesn't have a Javascript tag. If I click CSS, I get two items, but if I add Frontend, I get 5.

    Hope this helps!

    Marked as helpful
    1
    Chrisā€¢740
    @chrisbailey85Posted over 3 years ago

    @emestabillo thank you for your feedback, I got stuck on fixing the filter but I think I've managed to fix it.

    1

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