Jobs

Community feedback

• 

  Julianna Messineo•290

  @mathematiCode

  Posted 5 months ago

  I really loved how you implemented the FiltersContext with the addFilter and removeFromList functions included. I just learned context so I didn't think of grouping those functions together into a custom hook. I noticed that you used span tags for the filters but this would make it hard for keyboard users to click them if the only way to do it is with a mouse. Anything that needs to be clicked should always be a button, and then you can just remove the default styles. Unless it's a link to an external page, then you can just use an anchor tag.

  11

  Marked as helpful