GoogleChromeLabs/quicklink

Consider adding `rootMargin` to the Intersection Observer

Open

#14 aberto em 10 de dez. de 2018

Ver no GitHub
 (13 comments) (1 reaction) (0 assignees)JavaScript (431 forks)batch import
enhancementgood first issuehelp wanted

Métricas do repositório

Stars
 (10.861 stars)
Métricas de merge de PR
 (Mesclagem média 47d 12h) (4 fundiu PRs em 30d)

Description

Feedback from @surma:

I feel like by default you should add a rootMargin to the IO to trigger the load before something comes into the viewport.

So, rootMargin defaults to "0px 0px 0px 0px", meaning the intersection will be computed between the root element's unmodified bounds rectangle and the target's bounds. This change would be made to https://github.com/GoogleChromeLabs/quicklink/blob/master/src/index.mjs#L21

Guia do colaborador

Direção de pesquisa
Modifique a chamada ao construtor IntersectionObserver em src/index.mjs para aceitar um parâmetro rootMargin opcional, com o padrão '0px 0px 0px 0px', e atualize a assinatura da função e a documentação de acordo.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Funcionalidade
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.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptIntersection Observer API
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70