angular/components

Overlay is not destroyed while navigating using Browser's Back/Forward button (RouteReuseStrategy is used)

Fermée

#17 973 ouverte le 16 déc. 2019

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)TypeScript (6 650 forks)batch import
P3area: cdk/overlayhelp wanted

Métriques du dépôt

Stars
 (24 044 étoiles)
Métriques de merge PR
 (Merge moyen 3j 12h) (98 PRs mergées en 30 j)

Description

Reproduction

https://angular-hlrhkp-pzxidw.stackblitz.io/

Steps to reproduce:

  1. Click on the navigate to second page link.
  2. Click on Browser Back Button and Open the select box .
  3. Click on the browser forward button to navigate.

Expected Behavior

What behavior were you expecting to see? The overlay of the previous component should be destroyed, when the browser navigation buttons clicked

Actual Behavior

What behavior did you actually see? The overlay of the material persists even after the page is routed to next page

Environment

  • Angular:
  • CDK/Material: 5.21
  • Browser(s): Chrome
  • Operating System (e.g. Windows, macOS, Ubuntu): Windows

Guide contributeur

Direction de recherche
Investigate how Angular CDK overlay's lifecycle interacts with router navigation when RouteReuseStrategy is used. Look at the overlay ref counting mechanism in the OverlayRef and how it handles detach on route change. Check if there is a missing cleanup when the route is reused. Consider examining the following files: overlay ref.ts, overlay container.ts, and the RouterModule setup.
Stack technique
typescript
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
3-5 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Angular basicsAngular CDK OverlayRouteReuseStrategy
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
25