ejplatform / ej-server

EJ uses gamification and machine learning to promote discussions and participation in a virtual democracy

https://ejplatform.github.io/ej-server/

Other

38 stars 17 forks source link

[REVIEW] Animação do formulário de comentário e contador de caracteres #608

Closed fabiolamfleury closed 5 years ago

fabiolamfleury commented 5 years ago

Descrição

A implementação do formulário de comentário aparecer com animação fade, apenas depois do usuário clicar no botão de criar comentário, substituindo a parte de votar nos comentários. Também foi acrescentado o contador de caracteres no campo do mesmo formulário.

*Obs: foi alterado no model de comment o limite de caracteres que estava com 210, coloquei 252

Issues Relacionadas

resolves: #596 resolves: #585

Checklist

[x] Os commits seguem o padrão do projeto (Flake8 e afins)
[x] Os testes estão passando e cobrem as mudanças
[x] Marcou no título do pull request se ele é work in progress [WIP] ou se está pronto para revisão [REVIEW]

Imagens/Comentários

Animação de como ficou implementado

gif4

Telas do Invision

(Antes de clicar)

(Depois de clicar)

codecov[bot] commented 5 years ago

Codecov Report

Merging #608 into develop will increase coverage by 0.02%. The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #608      +/-   ##
===========================================
+ Coverage    81.82%   81.85%   +0.02%     
===========================================
  Files          121      121              
  Lines         3202     3207       +5     
  Branches       343      344       +1     
===========================================
+ Hits          2620     2625       +5     
  Misses         482      482              
  Partials       100      100

Impacted Files	Coverage Δ
src/ej_conversations/models/comment.py	`80.64% <ø> (ø)`	:arrow_up:
src/ej_conversations/forms.py	`100% <100%> (ø)`	:arrow_up:
src/ej_conversations/routes/conversations.py	`100% <100%> (ø)`	:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 33346c1...2a21e82. Read the comment docs.

fabiolamfleury commented 5 years ago

@ricardopoppi @Heloisecs peço que analisem esse PR, principalmente a questão da animação, que não sei se ficou de acordo com o esperado, obrigada :)

ricardopoppi commented 5 years ago

@fabiolamfleury ficou ótimo! Apenas observei na issue #611 que no envio de comentário, os indicadores abaixo (restantes e moderação) não atualizam

MatheusRich commented 5 years ago

@fabiolamfleury Bela implementação, parabéns :smile:! Acredito que clicar no botão de "Criar Comentário" para fechar o balão é pouco intuitivo. Percebi que o ícone muda, acho que o texto poderia mudar também. Outra possível implementação poderia ser clicar fora do balão para fechá-lo. Acho que cabe dar uma testada!

ricardopoppi commented 5 years ago

@MatheusRich @fabiolamfleury sobre mudar o texto, não tenho certeza, acho melhor a gente testar com os usuários. Sobre clicar fora do campo e ele também fechar (mesmo efeito do botão) acho que seria uma solução importante!

fabiolamfleury commented 5 years ago

@ricardopoppi implementado nesse último commit o clique fora do campo para fechar o formulário :)

@MatheusRich obrigada pela sugestão

ricardopoppi commented 5 years ago

Maravilha @fabiolamfleury vamos nos organizar hj a tarde pra soltar a Ada 1.2 incluindo esse PR

LeoSilvaGomes-zz commented 5 years ago

@fabiolamfleury Relacionado ao front está muito bom!!