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

Updated the marker display and tf plugins to not reset enabled namespaces/frames #988

Merged
merged 1 commit into from
Apr 7, 2016
Merged

Conversation

bponsler
Copy link
Contributor

@bponsler bponsler commented Apr 7, 2016

Updated the marker display and tf plugins to update the map of enabled namespaces and frames whenever those frames are enabled/disabled using the check boxes. Also updated the plugins so that the map of enabled namespaces and frames does not get erased whenever the plugin is reset.

This allows the currently selected namespaces/frames to remain selected after the Reset button is pressed.

…d namespaces and frames whenever those frames are enabled/disabled using the check boxes. Also updated the plugins so that the map of enabled namespaces and frames does not get erased whenever the plugin is reset.

This allows the currently selected namespaces/frames to remain selected after the Reset button is pressed.
@stuglaser
Copy link

I tried it out, and the behavior looks right to me. Thanks for the fix!

@wjwwood
Copy link
Member

wjwwood commented Apr 7, 2016

Thanks guys, lgtm me too.

@wjwwood wjwwood merged commit 005d8cc into ros-visualization:kinetic-devel Apr 7, 2016
@bponsler bponsler deleted the marker-tf-reset-fix branch April 7, 2016 22:37
wjwwood pushed a commit that referenced this pull request Apr 7, 2016
…d namespaces and frames whenever those frames are enabled/disabled using the check boxes. Also updated the plugins so that the map of enabled namespaces and frames does not get erased whenever the plugin is reset. (#988)

This allows the currently selected namespaces/frames to remain selected after the Reset button is pressed.
@wjwwood wjwwood mentioned this pull request Apr 7, 2016
wjwwood added a commit that referenced this pull request Apr 7, 2016
…d namespaces and frames whenever those frames are enabled/disabled using the check boxes. Also updated the plugins so that the map of enabled namespaces and frames does not get erased whenever the plugin is reset. (#988) (#989)

This allows the currently selected namespaces/frames to remain selected after the Reset button is pressed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants