miguel-perez/smoothState.js

storePageIn() fails to find container in new doc if nested

Open

#193 ouverte le 8 juil. 2015

Voir sur GitHub
 (8 commentaires) (0 réactions) (0 assignés)CSS (515 forks)batch import
bughelp wanted

Métriques du dépôt

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

Description

Nesting the container even one level under the body tag prevents storePageIn() from find it.

<body>
    <div>
        <article id="js-smoothstate-content">
            <h1>TEST CONTENT</h1>
        </article>
    </div>
</body>

The filter() function won't find the container id (Chrome v43):

storePageIn: function (object, url, doc, id) {
    /../
    object[url] = { // Content is indexed by the url
        /../
        html: $newDoc.filter('#' + id), // Stores the contents of the page
    }; 
}

Consider using .find() instead?

Guide contributeur

Direction de recherche
Le problème signale que storePageIn() utilise .filter() pour trouver l'élément conteneur, ce qui échoue lorsque le conteneur est imbriqué dans un autre élément. La correction suggérée est d'utiliser .find() à la place. Vérifiez le code source dans smoothState.js autour de la fonction storePageIn (probablement lignes 100-150) et assurez vous que changer $newDoc.filter('#' + id) en $newDoc.find('#' + id) résout le problème sans casser d'autres fonctionnalités. Assurez vous également que l'id est correctement délimité et qu'aucune autre partie du code ne dépend du comportement de .filter().
Stack technique
javascript
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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
basic JavaScriptjQuery
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80