[FEATURE]: Add automatic contrast for secondary hover labels
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 58/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- javascript
- Bereich
- data-visualization
Rechercherichtung
The issue points to Color.wcagContrast() and links PR #8115 and issue #2342. Read the review comment linked from PR #8115 and inspect how secondary hover labels get their text and background colors. Add automatic text color selection using a 4.5:1 contrast threshold, then verify that light trace colors produce readable labels while adequate contrast is preserved.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Description
Secondary hover labels inherit their text color from the trace color. However, light or low contrast trace colors can make the text difficult to read against the label background.
I propose using Color.wcagContrast() with a 4.5:1 threshold to automatically select a more readable text color.
Related to #2342 and #8115 (see the review comment: https://github.com/plotly/plotly.js/pull/8115/changes/1926941923cda4b90e4258e9e48e8c9702543083#r4383215739).
Before
After
- Vorherrschende Sprache
- JavaScript
- Sterne
- 18.4k
- Forks
- 2.1k
- Ø Merge
- 2 T. 16 Std.
- Gemergte PRs (30 T.)
- 11
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus plotly/plotly.js
-
[BUG]: `events` is required but not declared in dependenciesEvtl. vergeben @KoolADE85 hat das vor 6 Tagen übernommen. Offenbug P3 size: 1
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 83/100
plotly/plotly.js#8163 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
[BUG]: `sankey`: `node.x` / `node.y` of `0` is ignoredEvtl. vergeben @adimalkar hat das vor 11 Tagen übernommen. Offenbug P3 size: 1
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
plotly/plotly.js#8150 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
[BUG]: `scatterpolargl` drops points between 10000 and 100000 pointsEvtl. vergeben @Tanishdete333 hat das vor 11 Tagen übernommen. Offenbug P3 size: 1
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
plotly/plotly.js#8148 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
[BUG]: findEmpties throws "iterated with no new neighbors" for large NaN regions in contour (double-precision underflow)Evtl. vergeben @davrens101 hat das vor 10 Tagen übernommen. Offenbug P3 size: 3
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
plotly/plotly.js#8108 · 4 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
chore P3 plotly-internal size: 3 task
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 77/100
plotly/plotly.js#8064 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in plotly/plotly.js
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
SasanLabs/VulnerableApp#912 ·
Maintainer antworten meist innerhalb von 1 Tag
-
[chore]: Force minimatch 11.0.1 in the CI scripts' jest tooling (Dependabot #140)Evtl. vergeben @alishair7071 hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
jaegertracing/jaeger#10133 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
accessibility
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
OREL-group/Project-Management#133 · 1 Kommentar ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
volcengine/OpenViking#7485 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag