Add an option to change the highlighting background color of `updatemenus`
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
Currently only the normal (i. e. unhighlighted) background color can be changed through the parameter bgcolor. If you set this background to a dark coloring, you most probaby also want to adjust the text font to a bright coloring so that the text is still readable. But then the text becomes pretty unreadable while being highlighted since the highlighting background color is very bright!
Therefore I think an additional parameter bgcolor_active or the like should be introduced.
Additionally, an even more sophisticated solution could involve an automatic adjustment of the text font color and/or background color depending on the color brightness of the user defined bgcolor and/or font.color values (this seems to be already the case with the font color of hoverinfo labels).
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
Comienza con las entradas de referencia de layout updatemenus para bgcolor y font.color y, después, sigue el manejo existente del estado resaltado. Decide si la opción de fondo activo solicitada es el cambio acotado; se considera terminado cuando la opción está documentada y los controles resaltados siguen siendo legibles con fondos normales oscuros, con cobertura para el nuevo comportamiento.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- data-visualization
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100