Countries API Project

Community feedback

• 

  Gillian Akpeki•445

  @akpekigPosted over 1 year ago

  Refactoring feedback:

  • When the data initially loads from the API, styles are no longer applied. You have to click the dark/light mode button for styles to come back. This is worth investigating.

  • DRY (Don't Repeat Yourself) is a principle that would help with the styles. It's great you've got a light and dark mode implemented, but both files repeat a lot of code. Seeing as Sass is modular anyway, why not make a third file with all the repeated code that acts as a common module that's never removed?

  • SOLID principles would help out with the JS part. IMO, there's a lot of getting and setting that doesn't feel as modular as they should be.

  Marked as helpful

  1