Leaflet/Leaflet

Investigate and possibly resolve code duplication in `Circle`

Open

#10.046 geöffnet am 5. Jan. 2026

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (6.124 Forks)batch import
help wantedneeds investigationrefactoring

Repository-Metriken

Stars
 (45.018 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 43m) (1 gemergte PR in 30 T)

Beschreibung

Currently, Circle extends from CircleMarker, and overrides certain methods to use a custom _mRadius property for the radius. However, it seems that the logic is largely comparable to the super-class, which uses its own _radius property. In fact, both these properties are used within the Circle class, somewhat interchangeably.

As part of this issue, we need to determine if the logic in Circle is actually the same as CircleMarker and we can unify the _mRadius and _radius properties, and if so, unify them into a single property, eliminating redundant methods in the process (e.g. getRadius(), setRadius(), setStyle(), etc.).

Contributor Guide

Research-Richtung
Vergleichen Sie die Implementierung von Circle und CircleMarker, um doppelte Logik zu identifizieren, und erwägen Sie eine Refaktorisierung zur Beseitigung von Redundanzen durch Vereinheitlichung der Eigenschaften mRadius und radius.
Tech Stack
javascript
Domain
frontend
Issue Type
Refactoring
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 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptLeaflet
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60