refac: arrumei a bagunça, ou parte dela

[tulliorbr]

Ajustes:

• scroll nada a ver na tela
• funcionalidade dos botões faltantes, para fullscreen e editar (podendo editar somente em modo edição)
• funcionalidade de contagem regressiva funcionando
• dei uma arrumada, mas não está 100%, no relógio da contagem regressiva
• validei os inputs para não poder digitar qualquer coisa neles
• botei os zeros que faltavam para o timer n ficar baguela

Coisas que faltam mas fiquei compreguiça (ou pq o techlead me chamou para arrumar coisa em prod):

• cor do botão de cancelar quando está em contagem regressiva
• parar está sempre resetando o timer todo para 30 segundos ao inves de parar a contagem onde estava
• deve ter outras coisas, mas me atrapalhei e esqueci

Adoro o canal, um abraço a todos <3

[gabrnunes]

@tulliorbr acha que pode usar a #2 como base? pra gente ir melhorando em cima do outro código haha

[tulliorbr]

@gabrnunes acho que estavamos fazendo praticamente ao mesmo tempo as mesmas coisas, se eu rebasear minha branch na dele, provavelmente vou sobreescrever as coisas dele, será que não tem problema?

[gabrnunes]

@tulliorbr eu acho que o ideal era vc ver o que pode melhorar na sugestão que ele fez, tá ligado? ver quais alterações vc fez que ele não fez

[tulliorbr]

pronto @gabrnunes, parece que ficou certinho

[gabrnunes]

Boa, @tulliorbr assim que a dele mergear eu vejo aqui haha

[geanfarias]

boa!

[geanfarias]

@tulliorbr, vc fez o merge da branch dele na sua, mas não apontou o pr para o pr anterior

[tulliorbr]

Olha, realmente não entendi como deve ser feito, criei um PR agora no fork dele, abrindo um PR do meu fork para o dele
Mirand43#1
Mas não tenho certeza se deveria ser assim kkkkk
Se ainda não for isso, explica certinho o que devo fazer que vou arrumar <3

[geanfarias]

@tulliorbr, tá vendo que seu pr está apontando para a branch main? Vai ter um botão de Editar do lado do título do pr, ao editar você consegue alterar a branch para onde seu pr está apontando

[gabrnunes]

@geanfarias eu tentei fazer isso, mas como se tratar de branchs remotas, num dá pra fazer isso haha

mas acabei de mergear a outra PR, dessa forma deve corrigir os códigos duplicados

[tulliorbr]

@gabrnunes @geanfarias top demais então! Verifiquei após o merge do PR anterior e parece tudo certinho, sem conflitos, fico a disposição caso seja necessário corrigir algo após o code review

[WellingtonGuedesDev]

Estava fazendo convertendo os valores para segundos, assim a apenas precisa decrementar os segundos que a hora e os minutos iriam decrementar dinamicamente, mas vi que foi corrigido e não fiz o pull, mas é uma opção, fica mais clean.

let hourToSeconds = (parseInt(hours.value) * 60 * 60) + (parseInt(minutes.value) * 60) + (parseInt(seconds.value))

let hours = Math.floor(hourToSeconds / 60 / 60)
let minutes = Math.floor(hourToSeconds / 60 - (hours * 60))
let seconds = Math.floor(hourToSeconds %  60)