rest-countries-react

Community feedback

• 

  

  Grace•32,130

  @grace-snow

  Posted over 4 years ago

  Hi there

  Well done on this, it looks and functions well on my phone, and the code is nice to read and understandable, even when I don't know much React. Good job!

  I only looked at the theme switcher component and can suggest a small change to make it fully accessible. At the moment, it doesn't communicate to assistive tech which theme is being activated. If you added an aria-label attribute to the button, you could use a ternary to update the label content. Something like ariaLabel={theme === "dark" ? "change to light mode" : "change to dark mode"}

  Like I said, I don't do react, but I think that would work. There may be other places in your code that need similar, not sure.

  Anyway, really good work again. Well done!

  21
• 

  

  Chamu•13,860

  @ChamuMutezva

  Posted over 4 years ago

  Well done, most functionality are working well but I think the visual display still need a second look.

  • the border countries can be updated to read the full name of the country instead of an abbreviation.
  • take note also that some mobile phones automatically add an empty space at the end of a word when typing, that can be fixed by applying the trim method. Happy coding

  11