SOS-RS / frontend

Auxílio RS: Projetos de Resposta a Emergências por Chuvas e Alagamentos
https://sos-rs.com
MIT License
598 stars 333 forks source link

[BUG] Pagina sem monitoramento de scroll #265

Closed williandeitosi closed 1 month ago

williandeitosi commented 3 months ago

Descrição

A página não está retornando o valor correto do scroll mesmo quando o usuário rola verticalmente..

Problema encontrado

Quando a página é rolada verticalmente, o valor de window.scrollY não está sendo atualizado corretamente, retornando sempre 0 independentemente da posição de rolagem atual. Isso impede a funcionalidade que depende do valor do scroll, como a exibição de botao voltao ao topo que é a funcionalidade que eu estou tentando desenvolver.

Prioridade

Registre apenas a prioridade ou seriedade do defeito. Um guia:

Solução proposta

o valor de window.scrollY refletisse corretamente a posição de rolagem vertical da página.

Ambiente

Testado no navegador Google Chrome em um computador Windows 11.

Evidência

Captura de tela 2024-05-19 085443

allangalera commented 3 months ago

Pelo visto isso é causado pq o #root tem um overflow-x: hidden. Testei algumas páginas e não parece ter efeito nenhum quando eu removo 🤔

image

Alguem sabe se tem algum motivo pra esse overflow: hidden ter sido adicionado ou é seguro remover?

Isso impede a funcionalidade que depende do valor do scroll, como a exibição de botao voltao ao topo que é a funcionalidade que eu estou tentando desenvolver.

Concordo q seria mais fácil utilizar o métodos da window pra criar efeitos de scroll, porém isso não impede de criar funcionalidades de scroll. Sabendo que é a div com id #root é possível adicionar eventos de scroll sem problemas

Pra validar isso cole esse código abaixo no console do site em producção:

document.getElementById('root').addEventListener('scroll', (e) => console.log(e.target.scrollTop))