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

AnnotationManager::update doesn't update LineAnnotation / FillAnnotation #5844

Closed
tobrun opened this issue Aug 2, 2016 · 1 comment
Closed
Labels
Core The cross-platform C++ core, aka mbgl

Comments

@tobrun
Copy link
Member

tobrun commented Aug 2, 2016

I have been looking into implementing updatable Polyline/Polygons on the Android bindings.
For this I'm hooking into the following methods exposed in AnnotationManager:

    Update update(const AnnotationID&, const LineAnnotation&, const uint8_t);
    Update update(const AnnotationID&, const FillAnnotation&, const uint8_t);

When I execute the update method, I'm seeing the polygon being removed but not readded:

ezgif com-video-to-gif 23

The code used here is equivalent to the one used for SymbolAnnotation.

@tobrun tobrun added the Core The cross-platform C++ core, aka mbgl label Aug 2, 2016
@tobrun tobrun added this to the android-v4.2.0 milestone Aug 2, 2016
@tobrun
Copy link
Member Author

tobrun commented Aug 2, 2016

I'm currently working around this by calling remove/add myself instead of AnnotationManager. While this works it's doesn't have the correct ID since a new ID is assigned with add.

ezgif com-video-to-gif 26

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Core The cross-platform C++ core, aka mbgl
Projects
None yet
Development

No branches or pull requests

1 participant