Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

QueryRenderedFeatures should include source and sourceLayer #11846

Closed
asheemmamoowala opened this issue May 5, 2018 · 2 comments · Fixed by #15480
Closed

QueryRenderedFeatures should include source and sourceLayer #11846

asheemmamoowala opened this issue May 5, 2018 · 2 comments · Fixed by #15480
Assignees
@jmalanen
Labels
Core The cross-platform C++ core, aka mbgl GL JS parity For feature parity with Mapbox GL JS querying Issues with querySourceFeatures or queryRenderedFeatures

Comments

@asheemmamoowala
Copy link
Contributor

asheemmamoowala commented May 5, 2018 •
edited by lbud
Loading

In #6263(gl-js), Map#queryRenderedFeatures was updated to include the source, sourceLayer, and state properties on each feature in the results.

Renderer::queryRenderedFeatures() should be updated to include these properties in its results.
@asheemmamoowala asheemmamoowala added the GL JS parity For feature parity with Mapbox GL JS label May 5, 2018
@asheemmamoowala asheemmamoowala mentioned this issue Jul 18, 2018
Closed
@1ec5
Copy link
Contributor

1ec5 commented Jul 21, 2018

Currently, the native implementation’s feature querying results are plain features that also lack the style layer ID and evaluated layer properties: #7162.

@kkaefer kkaefer added Core The cross-platform C++ core, aka mbgl querying Issues with querySourceFeatures or queryRenderedFeatures and removed querying Issues with querySourceFeatures or queryRenderedFeatures labels Sep 25, 2018
@stale stale bot added the archived Archived because of inactivity label Mar 24, 2019
@stale
Copy link

stale bot commented Mar 25, 2019

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Mar 25, 2019
@stale stale bot removed the archived Archived because of inactivity label Mar 26, 2019
@jmalanen jmalanen self-assigned this Sep 13, 2019
@jmalanen jmalanen mentioned this issue Sep 18, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jmalanen jmalanen

Labels
Core The cross-platform C++ core, aka mbgl GL JS parity For feature parity with Mapbox GL JS querying Issues with querySourceFeatures or queryRenderedFeatures
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for feature state APIs mapbox/mapbox-gl-native
4 participants
@kkaefer @1ec5 @asheemmamoowala @jmalanen