Implementação do alerta de cheer #12
Conversation
narcello
changed the title
jlcarvalho
changed the title
|
A gente só tem que ver em relação a conflitos na hora de dar o merge na main @jlcarvalho @narcello. Mas boa demais!!! |
Sim sim, hora que aprovar o seu PR vai pipocar os conflitos aqui pr'eu resolver :) Tá massa, tá bonito!! |
|
Eu acabei mudando como os alertas funcionam pra permitir que o bot suporte múltiplos eventos de um mesmo tipo de forma simultânea (6cbc92b). Então vai ter bastante conflito 😄 Desculpa. |
@jlcarvalho Ta de boas :) resolvido |
| socket.on('cheer', function(msg) { | ||
| const cheerBox = createAlertBox(`Obrigado pelos ${msg.bits} bits ${msg.username}!!!`); | ||
|
|
||
| window.requestAnimationFrame(() => { |
There was a problem hiding this comment.
-
@jlcarvalho Precisa desse
requestAnimationFrame? Pois la dentro da funcão já tem um. -
Outra coisa. O
requestAnimationFramefunciona bem com timeout? Não deveria ter umrequestAnimationFramena para lidar comelement.style.opacity = '0';L-32 e outro paraelement.remove();L-36? Perguntando porque li a doc aqui mas nao achei nada sobre.
There was a problem hiding this comment.
1 -Esse requestAnimatonFrame tá aí pra "quebrar" o event loop do javascript em 2 partes (podíamos inclusive ter usado um setTimeout(() => {}, 0) para o mesmo propósito), mas assim, não lembro exatamente o motivo de ter colocado ele (talvez a animação não estivesse funcionando?). Podemos testar novamente amanhã na live.
2 - Funciona ok. A única diferença é que ele não vai esperar o próximo frame pra começar a fazer algo, mas durante a implementação nós não nos preocupamos com a performance da animação (até pq não tivemos nenhum problema de performance visível), só nos preocupamos em fazer ela funcionar como esperado.
Não testei, na real perdi umas live e não to sacando muito bem o código.
Me baseei no que já tinha pronto e no PR aberto do @Limatucano.
Closes #2