Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Namespace-scoped Marker DELETEALL action #684

Closed
hidmic opened this issue May 11, 2021 · 0 comments · Fixed by #685
Closed

Namespace-scoped Marker DELETEALL action #684

hidmic opened this issue May 11, 2021 · 0 comments · Fixed by #685
Labels
enhancement New feature or request

Comments

@hidmic
Copy link
Contributor

hidmic commented May 11, 2021

Feature request

Feature description

Current Marker and MarkerArray displays support DELETE actions, to drop markers one by one based on their ID, and DELETEALL actions, to all markers associated with that display. However, having a third delete variant, a namespace-scoped DELETEALL action, would greatly simplify applications that use many markers and namespace them based on some application-specific logic or concept to simplify the interaction (i.e. through visibility checkboxes).

Implementation considerations

This is an easy and ABI-compatible feature addition. All delete logic would be restricted to this switch case.

@hidmic hidmic added the enhancement New feature or request label May 11, 2021
@hidmic hidmic changed the title Namespace-scoped Marker DELETE_ALL action Namespace-scoped Marker DELETEALL action May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant