mapbox / mapbox/mapbox-maps-android

Implement source.filter from the MapBox styleJSON spec

Open
#2,700 1 comment 4 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

auto-triaged feature :green_apple:
Dominant language
Kotlin
Stars
578
Forks
161
PR merge metrics
No merged PRs in 30d

Description

New Feature

Add filtering at the source level without having to set the data attribute. This already exists as source.filter in the MapBox style spec.

Why

I would love to be able to filter sources without having to set the data attribute, and be able to filter before clustering (which layer-level filtering doesn't allow). mapbox-gl already implements this and it's super convenient.

Thank you :)

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the MapBox style specification linked in the issue and compare the requested source.filter behavior with mapbox-gl's implementation. Trace the source handling and clustering entry points in this Android project, then identify the relevant tests. Done means GeoJSON sources accept filtering before clustering without requiring the data attribute.

Written by the indexing model from the issue text.

Assessment

Tech stack
kotlin
Domain
mobile
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.