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

Hotfix: Alguns links de vagas não funcionam no Safari do Ipad #334

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

WillRy
Copy link
Contributor

@WillRy WillRy commented Dec 8, 2024

Olá a todos,

Anteriormente havia feito um pull request sobre uma correção de links de vagas não funcionarem corretamente no Safari do Iphone. O pull request resolveu o problema no Iphone, mas aparentemente no Ipad o problema ainda persiste.

PR anterior

Ao investigar o problema, encontrei inúmeros relatos que além da substituição do "window.open", também deve ser feito um tratamento para a abertura de links que são feitos dentro de chamadas assíncronas, como por exemplo: Dentro de um fetch, que é o caso do card de vagas, onde o link é aberto dentro de um fetch.

Existe evidências de outras pessoas no stack overflow com o mesmo problema/solução:

Evidência

Também gerei reproduções dentro do codepen, emulando um link sendo aberto dentro de um fetch:

Com o bug no IOS:

Com bug

Sem bug no IOS:

Sem bug

Ao colocar a abertura do link dentro de um setTimeout, o safari não bloqueia as operações de abertura de links, há pouca explicação disponível na internet sobre a causa do problema, mas acredito ser algo relacionado a callback queue.

Caso tenha algum dispositivo com o Safari, vai conseguir ver na prática o problema nos links do codepen que enviei

Fiz os testes em Iphone e Ipad, nas versões que estou utilizando funcionou, mas seria interessante um teste com mais alguém que tem dispositivos IOS (o que é difícil achar em abundância, devido ao custo nada barato huehuehue)

Desde já, muito obrigado

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