[CHORE]: remove `scatter` trace from custom bundle
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 18.3k
- Forks
- 2k
- Merge moyen
- 2 j 12 h
- PR mergées (30 j)
- 28
Description
Quoted from https://github.com/plotly/plotly.js/blob/main/CUSTOM_BUNDLE.md:
Please note that the scatter trace is currently included in all bundles and cannot be removed. This behaviour may change in the future, so we recommend that you explicitly include scatter anyway if you need it in your bundle.
There was previously an attempt to make scatter optional in #5535, but that PR was closed without being merged.
Since plotly.js is relatively large even when using a custom bundle, being able to exclude unused traces such as scatter would help further reduce the bundle size.
I'm not sure how much bundle size reduction this would result in, since scattergl, scatterternary and some other traces reuse parts of the scatter implementation. Still, it seems useful for custom bundles to avoid including trace modules that are not explicitly requested where possible.
Would it be worth to re-consider this optimization?
If scatter still needs to be included for architectural reasons, it may be worth updating the documentation? Since the PR is closed.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par CUSTOM_BUNDLE.md et la discussion fermée #5535 afin de comprendre la contrainte actuelle liée au bundling des traces scatter. Vérifiez si les bundles personnalisés peuvent exclure scatter sans casser scattergl, scatterternary ou d’autres traces dépendantes. Le travail est terminé si scatter non demandé est exclu sans risque ou si la documentation explique clairement pourquoi il doit rester inclus.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- build-system, data-visualization
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- Active
- Clarté
- À clarifier
- Accessibilité débutants
- 38/100