Line width = 0 in templates effectively disables hline and vline
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 18.8k
- Forks
- 2.8k
- Ø Merge
- 16 Std. 26 Min.
- Gemergte PRs (30 T.)
- 21
Beschreibung
Three of the templates, ggplot2, simple_white and seaborn have line width set to 0.
https://github.com/plotly/plotly.py/blob/743b52484f47c518b85a04027c1b393528f44c15/packages/python/plotly/templategen/definitions.py#L44
https://github.com/plotly/plotly.py/blob/743b52484f47c518b85a04027c1b393528f44c15/packages/python/plotly/templategen/definitions.py#L95
https://github.com/plotly/plotly.py/blob/743b52484f47c518b85a04027c1b393528f44c15/packages/python/plotly/templategen/definitions.py#L223
This causes the add_hline and add_vline methods to not appear by default when using these templates, and the line_width property has to be set manually. I'm proposing that these templates have a thin line width value to enable vlines and hlines by default.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in packages/python/plotly/templategen/definitions.py bei den referenzierten Zeilen für die ggplot2-, simple_white- und seaborn-Vorlagen. Prüfe, wie sich ihre Linienbreiten von null auf add_hline und add_vline auswirken, und verifiziere anschließend, dass eine dünne Standardlinie beide Methoden ermöglicht, ohne dass line_width manuell gesetzt werden muss.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- data-visualization
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 45/100