fin-hypergrid/core

No way of temporarily displaying a grid and removing it, as clean up isn't implemented

Open

#629 aberto em 16 de set. de 2017

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)JavaScript (148 forks)batch import
bughelp wanted

Métricas do repositório

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

Description

Hi,

Thanks for a great library! :-) I've created some proof-of-concepts at work using the older version.

Right now I'm looking to open a "widget" (terrible name I know) that contains a grid. The user can close the widget. Therefore, it needs to have its resources cleaned up.

grid.stopPaintThread() and grid.stopResizeThread() are not hooked up correctly and throw.

I can call grid.canvas.stopPaintLoop() successfully to stop hooking into requestAnimationFrame instead.

However, there is currently no way to remove the document level events that the canvas hooks up. This causes exceptions to be thrown once the element containing the div is removed from the page.

See: https://github.com/fin-hypergrid/core/blob/master/src/lib/Canvas.js#L53

Guia do colaborador

Direção de pesquisa
Inspecione o Canvas.js linha 53 e métodos relacionados para implementar uma função de limpeza que remova os ouvintes de eventos em nível de documento e corrija stopPaintThread/stopResizeThread.
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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
JavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60