Range slider emits relayout evt on mousemove, should be only on mouseup
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 18.3k
- Forks
- 2k
- Merge medio
- 2 d 12 h
- PR fusionados (30 d)
- 28
Descripción
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?
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza en rangeslider/draw.js, en mouseUp, y compáralo con la ruta de mousemove que emite eventos relayout. Traza el manejo de eventos y la cobertura existentes de rangeslider antes de decidir si el comportamiento deseado es un evento relayout modificado o un nuevo evento end. Se considera terminado cuando el comportamiento del evento coincide con el requisito de mouseup-only del issue.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- data-visualization, frontend
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100