fin-hypergrid/core

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

Open

#629 aperta il 16 set 2017

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)JavaScript (148 fork)batch import
bughelp wanted

Metriche repository

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

Descrizione

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

Guida contributor

Direzione di ricerca
Ispeziona Canvas.js riga 53 e i metodi correlati per implementare una funzione di pulizia che rimuova i listener di eventi a livello di documento e corregga stopPaintThread/stopResizeThread.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60