miguel-perez/smoothState.js

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

Open

#193 aperta il 8 lug 2015

Vedi su GitHub
 (8 commenti) (0 reazioni) (0 assegnatari)CSS (515 fork)batch import
bughelp wanted

Metriche repository

Star
 (4433 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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?

Guida contributor

Direzione di ricerca
Il problema segnala che storePageIn() utilizza .filter() per trovare l'elemento contenitore, il che fallisce quando il contenitore è annidato all'interno di un altro elemento. La soluzione suggerita è utilizzare .find() invece. Controlla il codice sorgente in smoothState.js intorno alla funzione storePageIn (probabilmente righe 100-150) e verifica che cambiare $newDoc.filter('#' + id) in $newDoc.find('#' + id) risolva il problema senza rompere altre funzionalità. Assicurati anche che l'id sia correttamente definito nell'ambito e che nessun'altra parte del codice dipenda dal comportamento di .filter().
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
basic JavaScriptjQuery
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80