Port feature state APIs to Native

@tobrun

GL-Native's separation of Style/Render-Style classes requires that the feature state APIs sync changes to the state across threads on platforms that implement async rendering. The port will be more complicated than the SourceCache based implementation in gl-js.

Native SDKs will need to expose feature state APIs on top of the core implementation, and support the feature-state expressions.

cc @tobrun @1ec5 @mollymerp @ansis

@fabian-guerra

Native SDKs will need to expose feature state APIs on top of the core implementation, and support the feature-state expressions.

For the iOS/macOS API, ideally the getter and setter would be added to MGLVectorTileSource, MGLShapeSource, and MGLComputedShapeSource. That would reduce the number of parameters needed for the setter.

Can the setter accept an MGLFeature returned from a feature querying method? Those features currently lack context such as the containing style layer or source layer: #7162. The GL JS implementation also accepts a “feature identifier” object containing a source, sourceLayer, and id, which would imply a method like -[MGLVectorTileSource setState:forFeatureWithIdentifier:sourceLayerIdentifier:].

The setter can accept a whole dictionary as state, which is quite unlike most state properties in Cocoa libraries that are typed as enumerations. The closest Cocoa analogy to the concept of “feature state” that I can think of would be the context parameter of -[NSExpression expressionValueWithObject:context:], which is associated with the variable syntax in NSExpression format strings. However, we’ve already repurposed that variable syntax for expression variables and for certain special expressions like $featureIdentifier.

/cc @fabian-guerra

@1ec5

@1ec5 I opened #11846 previously to add more context to features returned from the querying methods. If that context can be added to MGLFeature in some way, then that can be the input for the setState: method.

@jfirebaugh

@jfirebaugh any eta on this?

Looking forward to this one as well!

Hey Mapbox folks – Is there an updated ETA on this?

@JohnnyUX

Hey Mapbox folks – Is there an updated ETA on this?

Hi @JohnnyUX, we are working on reducing the gap between GL Native and GL JS (https://github.com/mapbox/mapbox-gl-native/blob/master/platform/node/test/ignores.json) but this is not the highest priority ATM because normally the use case has to do with hover a cursor over the feature which is not so common on mobile.

Adding a screenshot of feature state demo on mbgl-glfw app. The demo implements the same hover effect as this GL-JS example.

@tarigo

Assigning @tarigo for the android bindings

The core part has now been merged (PR #15480).

Any progress or planning/estimate for this feature? I hope this can be ported to iOS / Android as it is a really nice way to style selection states on top of external data.

It's been two years since this request was filed. Can we get an update?

Hi, another seven months passed, would you be able to give us some insight into your roadmap for this, please? This feature would make things so much easier on mobile!

This feature is available on the v10.0.0 of the mobile SDKs:

Public API for Android implementation of MapboxMap#setFeatureState:

  /**
   * Update the state map of a feature within a style source.
   *
   * Update entries in the state map of a given feature within a style source. Only entries listed in the
   * \p state map will be updated. An entry in the feature state map that is not listed in \p state will
   * retain its previous value.
   *
   * Note that updates to feature state are asynchronous, so changes made by this method migth not be
   * immediately visible using getStateFeature().
   *
   * @param sourceId Style source identifier.
   * @param sourceLayerId Style source layer identifier (for multi-layer sources such as vector sources).
   * @param featureId Identifier of the feature whose state should be updated.
   * @param state Map of entries to update with their respective new values.
   */
  fun setFeatureState(
    sourceId: String,
    sourceLayerId: String?,
    featureId: String,
    state: Value
  )

on iOS it's availlable as mapView.__map.setFeatureState(..)

Navigation Menu

Repository navigation

Port feature state APIs to Native #12413

8 remaining items

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Port feature state APIs to Native #12413

Description

Activity

1ec5 commented on Jul 18, 2018

asheemmamoowala commented on Jul 18, 2018

guyinn commented on Aug 2, 2018

cammanderson commented on Sep 26, 2018

john-x-u commented on Mar 4, 2019

tmpsantos commented on Mar 26, 2019

8 remaining items

jmalanen commented on Sep 4, 2019

tobrun commented on Sep 9, 2019

jmalanen commented on Sep 18, 2019

mattijsf commented on May 7, 2020

LocusLabsGit commented on Aug 17, 2020

sedot42 commented on Mar 29, 2021

tobrun commented on Mar 29, 2021

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions