-
Notifications
You must be signed in to change notification settings - Fork 91
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
feat(HideAnnotation): Hide button added to annotations to make them q… #502
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The button works but there needs to be an similarly easy method to turn them back on. I also assume that the checkbox doesn't represent the "Show Annotation" state because it is also considering whether or not the geometry is shown? @schmidtk ?
Annotations don't currently show on geometries (also happens on tracks - #353), so for now the checkbox represents the shown state of the annotation - we need to decide on if the annotation should be tied to the geometry or a separate entity. |
KML nodes that support showing an annotation should have a show/hide annotation button in their Layers window UI to toggle the annotation on/off. We still need to work on which nodes support annotations (see the linked track ticket), so we don't need to tackle that here. That button should control the When the If you need a way to notify the node that the option has changed, you can fire a |
…uickly removeable.
#495