plotly / plotly/plotly.js

Add a classname containing UID to barlayer traces too (not only scatter ones)

Ouverte
#7,328 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

community feature P2
Langage dominant
JavaScript
Étoiles
18.3k
Forks
2k
Merge moyen
2 j 12 h
PR mergées (30 j)
28

Description

Currently, scatter traces have a classname containing their UID, of the form trace{UID}, but bar traces don't:
Image

Desired behavior

  • Also add the classname containing their UID:
    Image
  • Or add it for both bars & scatter as their ID: id="trace-{UID}":
    Image

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par localiser l’endroit où les traces scatter reçoivent leur classname basé sur l’UID, puis comparez ce chemin avec le rendu des traces bar. Utilisez les exemples DOM de l’issue pour vérifier que le classname choisi ou le shared id apparaît sur les éléments bar et scatter, puis confirmez que les deux types de traces exposent l’UID de manière cohérente.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
d3, javascript
Domaine
data-visualization
Type d'issue
Fonctionnalité
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.