Range slider emits relayout evt on mousemove, should be only on mouseup
Nessuno ha ancora preso questa issue.
- Lingua principale
- JavaScript
- Stelle
- 18.3k
- Fork
- 2k
- Merge medio
- 2g 12h
- PR unite (30g)
- 28
Descrizione
I have had the need to do something after the rangeslider had changed (update the min/max somewhere else on the page). I did not find any event that fired only after the slider had been released (a.k.a. mouseup) so I changed the rangeslider/draw.js:
function mouseUp() {
dragCover.removeEventListener('mousemove', mouseMove);
dragCover.removeEventListener('mouseup', mouseUp);
Lib.removeElement(dragCover);
// --> new code
gd.emit('plotly_rangesliderend', {
axis: axisOpts,
opts: opts
// --> end new code
});
Is this anything that would be merged? Should I do a PR?
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia in rangeslider/draw.js, in corrispondenza di mouseUp, e confrontalo con il percorso mousemove che emette eventi relayout. Analizza la gestione degli eventi e la copertura esistenti di rangeslider prima di decidere se il comportamento desiderato sia un evento relayout modificato o un nuovo evento end. Il lavoro è completato quando il comportamento dell’evento corrisponde al requisito mouseup-only dell’issue.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript
- Ambito
- data-visualization, frontend
- Tipo di issue
- Funzionalità
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100