NativeScript / NativeScript/angular
Modal route issue - route runs twice when returning from second modal containing route outlet
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- TypeScript
- Estrelas
- 55
- Forks
- 13
- Merge médio
- 16min
- PRs com merge (30d)
- 1
Descrição
This is most evident in iOS. I've set up a demo app:
https://stackblitz.com/edit/nativescript-stackblitz-templates-vwcmqt
In iOS, if you follow the steps (they're numbered) you'll see that when you navigate back on step five, the animation runs twice—backwards then forwards. This appears to come from the shouldDetach method NSRouteReuseStrategy. If I use the previous code for shouldDetatch that was altered here:
https://github.com/NativeScript/angular/commit/bebcde23c4d6a392638c23ec1e2bc63b06a9b43e
then it works without problem.
NOTE: For this demo, I've used 14.2.6-alpha.0 because without it, I can't use modals with ModalDialogParams because of injector scope issues. However, this router problem described above also occurs in 14.2.5 when the modal corrections are in place. Either way, something isn't working as it should.
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece pelo método NSRouteReuseStrategy.shouldDetach e compare-o com a implementação anterior no commit beb cde23c4d6a392638c23ec1e2bc63b06a9b43e. Reproduza o fluxo numerado na demo do StackBlitz vinculada no iOS, incluindo o retorno do segundo modal que contém um route outlet. A tarefa estará concluída quando a navegação para voltar da etapa cinco executar sua animação uma vez, em vez de para trás e para frente.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- angular, typescript
- Domínio
- frontend, mobile-dev
- Tipo de issue
- Bug
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 35/100