miguel-perez/smoothState.js

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

Open

#193 aberto em 8 de jul. de 2015

Ver no GitHub
 (8 comments) (0 reactions) (0 assignees)CSS (515 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (4.433 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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?

Guia do colaborador

Direção de pesquisa
O problema relata que storePageIn() usa .filter() para encontrar o elemento container, o que falha quando o container está aninhado dentro de outro elemento. A correção sugerida é usar .find() em vez disso. Verifique o código fonte em smoothState.js em torno da função storePageIn (provavelmente linhas 100-150) e confirme que alterar $newDoc.filter('#' + id) para $newDoc.find('#' + id) resolve o problema sem quebrar outras funcionalidades. Também garanta que o id esteja corretamente escopado e que nenhuma outra parte do código dependa do comportamento de .filter().
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
basic JavaScriptjQuery
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80