Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix lessons menu not scrolling on mobile #17

Merged

Conversation

ericknathan
Copy link
Contributor

Add fixed height and overflow auto to fix lessons menu scroll on mobile devices.

@ericknathan ericknathan changed the title fix: lessons menu not scrolling on mobile (#12) Fix lessons menu not scrolling on mobile Sep 23, 2023
@BernardoSM
Copy link
Contributor

Fala @ericknathan beleza?

Infelizmente isso só funciona quando você testa a versão mobile usando no desktop, se tu testar no celular não funciona...

Sugiro que instale o ngrok para fazer o tunneling e testar no mobile.

Provavelmente você terá que usar a biblioteca useScrollLock e travar o scroll do html/body quando o menu tiver aberto, daí provavelmente vai funcionar.

@ericknathan
Copy link
Contributor Author

Opa @BernardoSM, tudo tranquilo e você?

Inicialmente tentei fazer o uso da sua sugestão, o useScrollLock para a trava do scroll no body, porém o bug ainda continuava, talvez eu tenha feito da maneira incorreta, não sei haha

use-scroll-lock.mp4

Para conseguir corrigir fiz a utilização da dynamic viewport dvh ao invés de vh, assim ele parou de considerar o scroll da altura da barra superior de pesquisa dos navegadores do mobile.

dvh.mp4

Aliás tinha ouvido falar do ngrok mas nunca usei ele na prática de fato, curti bastante o propósito no qual pude aproveitar dele :)

@BernardoSM
Copy link
Contributor

Sua solução ficou melhor ainda! Mandou demais, obrigado pela contribuição :D

@BernardoSM BernardoSM merged commit 3dc514d into menthorlabs:main Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants