erikringsmuth/app-router

core-animated-pages multiple paths match same app-route

Open

#44 ouverte le 5 déc. 2014

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)HTML (83 forks)batch import
bugdesign discussionhelp wanted

Métriques du dépôt

Stars
 (608 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

This is an extension of PR https://github.com/erikringsmuth/app-router/pull/43 from @sohjsolwin.

Example router.

<app-router core-animated-pages transitions="cross-fade-all">
  <app-route path="/first" element="first-page"></app-route>
  <app-route path="/second" element="second-page"></app-route>
  <app-route path="/page/:pageNum" element="third-page"></app-route>
</app-router>

Paths /page/1 and /page/2 will both match the last route. The transition between these two pages won't be animated because setting coreAnimatedPages.selected = 3 doesn't change the page.

This may require a big re-think of the app-router / core-animated-pages implementation...

Guide contributeur

Direction de recherche
Examinez la logique de correspondance des routes dans app router pour comprendre comment plusieurs routes peuvent correspondre au même chemin, et concevez une méthode pour mettre à jour l'index sélectionné de core animated pages afin que les transitions fonctionnent correctement.
Stack technique
htmljavascript
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.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Web ComponentsRouting concepts
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30