Revisit Plotly.js compatibility with strict CSP
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
https://github.com/plotly/plotly.js/pull/7109 was added in to make Plotly.js work better in strict CSP environments (specifically, ones which disallow inline CSS); however, it seems some of that work was broken by the migration to ESBuild.
For example, we no longer produce a separate plotly.css file (see recent comments on that PR).
@camdecoster recently added #7527 which makes it easier to test different CSPs in the devtools.
There's at least one obvious issue when inline styles are disallowed, which is that the modebar button styling is completely broken:
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 revisando PR #7109 y sus comentarios recientes; después, inspecciona la migración a ESBuild y la configuración de pruebas de CSP del issue #7527. Reproduce el fallo de estilo del botón de modebar cuando no se permite CSS inline y comprueba si se genera un archivo plotly.css independiente. Se considerará completado cuando los entornos con CSP estricto conserven un estilo de modebar funcional y se verifique la salida de build relevante.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- data-visualization, security
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100