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(scrolling): bug on iOS Safari #604

Merged
merged 2 commits into from
Jul 31, 2022
Merged

Conversation

aprendendofelipe
Copy link
Collaborator

Tentativa de resolver o problema reportado em #572

Digo tentativa, pois não consegui reproduzir, então preciso que alguém que esteja enfrentando o problema realize os testes.

Imagino que seja algo específico do iOS (ou iOS+Safari) em conjunto com o ByteMD. O que pode estar ocorrendo é o ByteMD não está ocupando toda a altura que deveria para renderizar todo o conteúdo, o que, pelo overflow estar setado como auto, autoriza o navegador a rolar somente o box que está sendo tocado.

Se for isso, provavelmente essa alteração irá resolver, pois toda a tela será rolada e o viewMode do ByteMD deve ir aumentando sua altura antes de entrar no campo de visão.

Peço atenção para quem for testar, se não está sendo cortado algo no final do conteúdo.

overflow: hidden era a nossa primeira opção mesmo, mas como o overflow: auto estava funcionando bem, optamos por ele, já que poderia surgir alguma situação em que fosse necessária a rolagem, mas no caso do body é melhor deixarmos esse controle com o ByteMD.

@vercel
Copy link

vercel bot commented Jul 29, 2022

@aprendendofelipe is attempting to deploy a commit to the TabNews Team on Vercel.

To accomplish this, @aprendendofelipe needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

@filipedeschamps
Copy link
Owner

@aprendendofelipe uma dúvida: as implementações que fizemos de overflow era para evitar o problema do Zalgo somente? Ou tinham outros motivos?

Pergunto isso, pois se for apenas por conta do Zalgo, eu removeria qualquer controle de overflow e deixaria o comportamento anterior. Isso só no Content, porque no ContentList manteria o que foi feito 🤝

@aprendendofelipe
Copy link
Collaborator Author

@aprendendofelipe uma dúvida: as implementações que fizemos de overflow era para evitar o problema do Zalgo somente? Ou tinham outros motivos?

Pergunto isso, pois se for apenas por conta do Zalgo, eu removeria qualquer controle de overflow e deixaria o comportamento anterior. Isso só no Content, porque no ContentList manteria o que foi feito 🤝

No Content o overflow foi inserido no title e no body:

  • No caso do title, além do Zalgo, tem o problema dos textos longos sem espaços não serem quebrados independentemente da opção wordWrap.

  • No body foi só pelo problema com texto Zalgo. Esse é o principal problema, pois não depende só do autor do conteúdo, já que qualquer comentário poderia se sobrepor aos outros.


Peço atenção para quem for testar, se não está sendo cortado algo no final do conteúdo.

Talvez essa atenção que eu pedi tenha assustado, mas é bem pouco provável que dê algum problema de cortar parte do conteúdo. Mas, se o improvável ocorrer, então já vamos agir com antecedência, pois será um sinal de que teremos problemas mesmo deixando o default overflow: visible. E nesse caso improvável vamos ter que analisar melhor o comportamento do ByteMD no iOS para corrigir isso.

@vercel
Copy link

vercel bot commented Jul 31, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
tabnews ✅ Ready (Inspect) Visit Preview Jul 31, 2022 at 10:27PM (UTC)

@filipedeschamps
Copy link
Owner

Talvez essa atenção que eu pedi tenha assustado

hhahaha sim! Mas testei e ta tudo 100% 😍 fiquei um bom tempo tentando clicar e reproduzir o erro, mas não teve mais jeito 🤝

@filipedeschamps filipedeschamps merged commit 40115f2 into main Jul 31, 2022
@filipedeschamps filipedeschamps deleted the scrolling-bug-on-iOS-Safari branch July 31, 2022 22:33
@filipedeschamps
Copy link
Owner

Merged!!! Let's gooooo!!!

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