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

Symbol not hiding symbols below on mobile #10973

Closed
Jukurrpa opened this issue Jan 20, 2018 · 4 comments
Closed

Symbol not hiding symbols below on mobile #10973

Jukurrpa opened this issue Jan 20, 2018 · 4 comments

Comments

@Jukurrpa
Copy link

Jukurrpa commented Jan 20, 2018

Platforms: Android 5.3.1 & iOS 3.7.2

Steps to reproduce:

  1. Use two symbol layers, the one below with some symbols here and there. All placement options (allow icon overlap, etc) are off on both layers
  2. Add a symbol on the top layer at the same location as one from the bottom layer

The newly added symbol should make the nearby symbols of the layer below disappear, but it doesn't. This works properly in Mapbox Studio and on the web, but on both mobile versions the symbols stay. The issue also exists when just toggling the visibility of the top layer.

In our use case when the user clicks on a symbol, we add another one add the same location on a layer above.

Expected behavior (result in Mapbox Studio and web):

studio-hidden
studio-visible

Issue (on Android, similar on iOS):

mobile-hidden
mobile-visible

As you can see the original pin stays visible on mobile. Both mobile and web used the same style, source and data in this example (text difference is due to the time at which the screenshots were taken). The newly added symbol has a different id, bottom layer uses a vector source, top layer uses a GeoJson source.

@tobrun
Copy link
Member

tobrun commented Jan 22, 2018

@Jukurrpa we recently added symbol placement that takes in account multiple layers in #10468. This is aimed to be released with the 6.0.0 release on Android and will fulfil the use-case shown in OP. The first 6.0.0-beta release is planned for the week/early next. (fwiw you can test this behaviour out using the 6.0.0-SNAPSHOT build). Let us know if you have any more questions!

@tobrun tobrun closed this as completed Jan 22, 2018
@Jukurrpa
Copy link
Author

Thanks @tobrun . I take it the change will also be present in the next major release on iOS?

@friedbunny
Copy link
Contributor

@Jukurrpa Yup, the same thing goes for iOS — this is planned for v4.0.0.

@Jukurrpa
Copy link
Author

Perfect, thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants