search

loopinfinito / setup

Loop Infinito's Setup website
http://setup.loopinfinito.com.br
23 stars 11 forks source link

Bug no mouse over/out na listagem dos convidados da home #50

Closed almirfilho closed 8 years ago

almirfilho commented 9 years ago

Na branch refactor, o script que detecta os movimentos do mouse na lista dos convidados da home está bugado.

Não importa por onde o mouse entre ou saia, pois o nome do convidado sempre aparece e desaparece pelo lado direito. Verifiquei que o método getMouseEnterDirection sempre retorna 1, que corresponde à posição right no código, mas consegui descobrir a causa do problema. Na branch master está tudo ok, e a diferença entre os scripts das duas branches é apenas que removi muito lixo na refactor, e não modifiquei em nada o antigo script que já fazia essa detecção do mouse.

Podes dar uma olhada nisso, @caiogondim?

caiogondim commented 9 years ago

Prefiro retirar essa opção, pois não dá pra portar ela pra mobile. Que acha de pensarmos em um design pra mobile e deixar esse mesmo design pra desktop?

almirfilho commented 9 years ago

Subi pra staging. Dá uma olhada como ficou o mobile lá e me fala o que tu acha. Eu particularmente gosto dessa firula na home (desktop) e não vejo problema se ficar diferente do mobile, não acho que fica inconsistente saca. Mas visualiza ai e me fala. Ficou lindão S2:

caiogondim commented 9 years ago

O que não gosto é de não ter o nome do cara no mobile, saca? Só a foto não diz nada pra maioria das pessoas.

Prefiro ter o nome sempre visível.

O que tu acha?

2015-07-14 14:58 GMT+02:00 Almir Filho notifications@github.com:

Subi pra staging http://staging.setup.loopinfinito.com.br. Dá uma olhada como ficou o mobile lá e me fala o que tu acha. Eu particularmente gosto dessa firula na home (desktop) e não vejo problema se ficar diferente do mobile, não acho que fica inconsistente saca. Mas visualiza ai e me fala. Ficou lindão S2:

— Reply to this email directly or view it on GitHub https://github.com/loopinfinito/setup/issues/50#issuecomment-121231832.

almirfilho commented 9 years ago

Exatamente, por isso que to falando pra você olhar em staging. Veja como está lá ;)

caiogondim commented 9 years ago

Investigando.

caiogondim commented 9 years ago

Não é preguiça de debugar. Mas to começando a achar essa feature um pouco ruim.

Não seria melhor termos um layout como em mobile, com o nome sempre visível, inclusive com empresa do entrevistado? No scroll o comportamento dessa feature tá bem zuada também quando o mouse está em cima do container dos entrevistados.

almirfilho commented 9 years ago

Beleza, vou fazer uns experimentos sobre como ficaria no desktop, então =)