REST Countries API with color theme switcher built w/ Angular 🅰️

Community feedback

• 

  Bernardo Poggioni•5,360

  @R3ygoskiPosted about 1 month ago

  Olá Daniel, parabéns pela solução, ela está muito bem-feita.

  Notei algumas coisas no seu projeto que gostaria de pontuar. Vou começar pela parte de estilização.

  Quando se pesquisa apenas um país, por exemplo "Antarctica" e muda para o "Dark mode", acaba ficando uma parte branca na parte inferior da tela, isso está acontecendo devido a sua <div> filha do <app-root> não ter o mínimo de tamanho para min-height: 100vh. Me refiro a esse elemento: Elemento - Linha 27

  E uma outra coisa que notei é na parte lógica, se você tentar acessar a "Antarctica" você verá que não irá aparecer nada e resultará num erro no console. Acredito que isso está ocorrendo, pois, a Antártida não retorna alguns dados, então tem que ser feito um tratamento de dados para que não dê esse erro.

  E quanto ao comentário acima, realmente o Search não está funcionando de maneira correta, parece que ele não é acionado quando removemos alguma letra, por exemplo, digite "brazil" e depois apague até que fique só "bra", o certo era aparecer Brazil e Gibraltar, mas está aparecendo somente o Brazil.

  E é isso, parabéns seu projeto ficou muito bom.

  Marked as helpful

  1

  ---

  

  Daniel 🛸•44,270

  @danielmrz-devPosted about 1 month ago

  @R3ygoski Valeu pelo feedback, Bernardo.

  Estou ciente desses bugs, tentei resolver antes de postar, mas não consegui. Ainda tô aprendendo como o Angular realiza determinadas ações e esse foi o melhor que consegui até o momento.

  Vou refatorar esse projeto em breve e vou levar seu comentário em consideração. Obrigado!

  1
• 

  Saul Gabriel Aranguren•320

  @saularangurenPosted about 1 month ago

  Hi Daniel, your project is really impressive, however, there are some details missing, such as the search engine.

  1

  ---

  

  Daniel 🛸•44,270

  @danielmrz-devPosted about 1 month ago

  @saularanguren what do you mean by search engine?

  0