miguel-perez/smoothState.js

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

Open

#193 geöffnet am 8. Juli 2015

Auf GitHub ansehen
 (8 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)CSS (515 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (4.433 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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?

Contributor Guide

Research-Richtung
Das Problem meldet, dass storePageIn() .filter() verwendet, um das Containerelement zu finden, was fehlschlägt, wenn der Container in einem anderen Element verschachtelt ist. Der vorgeschlagene Fix ist, stattdessen .find() zu verwenden. Überprüfen Sie den Quellcode in smoothState.js um die Funktion storePageIn herum (wahrscheinlich Zeilen 100-150) und stellen Sie sicher, dass das Ändern von $newDoc.filter('#' + id) zu $newDoc.find('#' + id) das Problem löst, ohne andere Funktionen zu beeinträchtigen. Stellen Sie außerdem sicher, dass die ID korrekt begrenzt ist und keine anderen Teile des Codes auf das .filter() Verhalten angewiesen sind.
Tech Stack
javascript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
basic JavaScriptjQuery
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80