AnSavvides/d3act

Bubble chart text min value

Open

#29 geöffnet am 1. März 2017

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (300 Stars) (26 Forks)batch import
enhancementhelp wanted

Beschreibung

The bubble chart needs to have a value bigger than 15 to display text. This value is too random, should not this be a prop?

https://github.com/AnSavvides/d3act/blob/master/src/components/BubbleChart.js#L15

Contributor Guide

Tech Stack
javascriptreact
Domain
frontenddata
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
basic React knowledgefamiliarity with props
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80
Research-Richtung
Locate src/components/BubbleChart.js line 15 to find the hardcoded threshold value 15. Determine how to add a new prop (e.g., minTextValue) with a default value of 15. Ensure the prop is documented and added to propTypes. Test that the chart uses the prop instead of the hardcoded value.