mapbox / mapbox/mapbox-maps-flutter

Decoding FillLayer with fill expression makes layer transparent, others too

Offen
#968 2 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Dart
Sterne
380
Forks
204
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

when using `FillLayer.decode` function and having style json with fill color expression, the function sets fillColorExpression properly, but also sets fillColor to `0` - this causes layer to be transparent

It's similar for LineLayer when we have lineColor, or when we have textColor.
Also in some cases if the value is not an expression but the string direct value, then parsing / decoding throws error, because it tries to cast the value to List (but it can be just string)

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

Finde die Implementierung von FillLayer.decode und die entsprechenden Decodierungspfade für LineLayer lineColor und textColor. Reproduziere die gemeldeten Fälle für Ausdrücke und direkte Strings und füge anschließend eine Regressionstestabdeckung hinzu, die zeigt, dass Ausdruckswerte die Farbe nicht ebenfalls zurücksetzen und dass direkte String-Werte ohne Fehler decodiert werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
dart, flutter
Bereich
mobile
Issue-Typ
Bug
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.