plotly / plotly/plotly.js

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

Offen
#7,328 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

community feature P2
Vorherrschende Sprache
JavaScript
Sterne
18.3k
Forks
2k
Ø Merge
2 T. 12 Std.
Gemergte PRs (30 T.)
28

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, zu ermitteln, wo scatter traces ihre auf UID basierende classname erhalten, und vergleiche diesen Pfad mit dem Rendering von bar traces. Verwende die DOM-Beispiele des issue, um zu überprüfen, dass die gewählte classname oder shared id auf bar- und scatter-Elementen erscheint, und bestätige anschließend, dass beide trace-Typen die UID konsistent bereitstellen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
d3, javascript
Bereich
data-visualization
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.