Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[US24] Loading Spinners #149

Closed
6 tasks done
caiooliv opened this issue May 28, 2019 · 1 comment
Closed
6 tasks done

[US24] Loading Spinners #149

caiooliv opened this issue May 28, 2019 · 1 comment
Assignees
Milestone

Comments

@caiooliv
Copy link
Collaborator

caiooliv commented May 28, 2019

Descrição:

Alguns processos dentro da aplicação necessitam de requisições às API's e isso pode demorar algum tempo, é necessário um feedback ao usuário de que aquela operação está sendo feita, para isso deve se usar os load spinners, já presente em algumas operações dentro da aplicação.

Tarefas

Lista de tarefas para serem realizadas.

  • Adicionar spinner na tela de login.
  • Adicionar spinner na tela de registrar usuário.
  • Adicionar spinner na tela de edição de perfil.
  • Adicionar spinner na tela de atualização do feed.
  • Adicionar spinner na tela perfil.
  • Adicionar spinner na tela de card expandido

Comentários

@caiooliv caiooliv added this to the Sprint 10 milestone May 28, 2019
@LucasSiqz LucasSiqz changed the title [US24] Load Spinners [US24] Loading Spinners May 29, 2019
@RCristyan
Copy link
Collaborator

O que demandou mais tempo na conclusão dessa issue foi corrigir o posicionamento dos loading spinners nos componentes. Os spinners e as outras coisas da aplicação estavam em um mesmo grid, ou seja, mover a posição dos spinners também moveria a posição das outras coisas, o que seria errado.

Para corrigir este problema eu alterei o posicionamento do grid externo com marginTop e marginLeft para organizar o spinner e, assim que o spinner parasse de ser renderizado, reverti as alterações no grid interno. Desta forma foi possível fazer o spinner ficar no local correto sem alterar a página.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants