Issue #54, Inclusão de exit popup e animação do botão Alugar.

pokemongovet / pokemongovet.github.io

Projeto para incentivar adoção de animais pessoas que amam pokemons virtuais

https://pokemongovet.github.io/

Other

102 stars 38 forks source link

Issue #54, Inclusão de exit popup e animação do botão Alugar. #56

Closed hsouza90 closed 7 years ago

hsouza90 commented 7 years ago

Explicações comentadas no Issue #54.

fititnt commented 7 years ago

@evansantos @Dkmister @anabastos podem me ajudar a revisar esse PR aqui?

fititnt commented 7 years ago

Que bacana!

Revisei aqui. Duas recomendações aqui. Caso você tenha problema com elas, me avisa que a gente da merge e resolve isso no master pra não te tomar tempo.

Dos botões que se movem, deixar apenas o "adotar"
O botão de popup aparece, mas parece que só aparece na segunda vez que movo o mouse para fora da tela (Versão 51.0.2704.79 Ubuntu 14.04 (64-bit))*
Talvez seja um bug isolado

hsouza90 commented 7 years ago

Vou dar uma olhada, estou sem o computador padrão então estou usando um netbook com Slax.

Não me atentei a alguns detalhes, vou até ver se melhoro algumas coisas.

Não tenho tanta prática mas vamos aprendendo rs.

Entre hoje e amanhã deve estar pronto, quem tiver algum report que possa ajudar é só comentar.

See ya~

fititnt commented 7 years ago

Então deixa que a @rcgivisiez ou eu revemos isto depois do merge.

hsouza90 commented 7 years ago

Estou realizando algumas alterações, tinha um bug mesmo no popup (sendo implementado outro modelo), já devo finalizar hoje isso, já realizei a correção no botão e já inclui o contador regressivo "falso".

No caso eu fecho esse PR e envio outro com as atualizações ou da pra alterar nesse mesmo?

fititnt commented 7 years ago

Olha, se voce ja fez alteracao, e o PR ainda nao foi aceito, pode alterar nele mesmo

fititnt commented 7 years ago

Ficou ainda melhor.

Como falei em outro lugar, se quiser entra la em https://pgovet.rocket.chat/channel/site-principal e se tiver interesse.

Por mim podem dar merge. Só mais alguem dar o ok. O popup so parece que tem um z-index mais baixo o do que um botão e uma das linhas da tabela, mas isso pode ser resolvido direto no master

evansantos commented 7 years ago

@blackmoral a única coisa que identifiquei é problema de z-index do modal ficando abaixo dos demais elementos, de resto ficou fillet 😄

icoffani commented 7 years ago

Obrigada pela contribuição @blackmoral! Seu PR foi aceito. :smiley: