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

Add checkbox group for settings dialogs #44

Closed
wants to merge 3 commits into from
Closed

Add checkbox group for settings dialogs #44

wants to merge 3 commits into from

Conversation

adwilson10
Copy link
Contributor

This adds the class CheckBoxGroup to qt_gui_py_common for use in the SimpleSettingsDialog class. Options are not exclusive and return an array of selected indexes and selected options when polled using the get_settings function.

This class will be used to add optional plotting markers to rqt_plot and the rqt_bag plotting plugin.

@@ -0,0 +1,71 @@
# Software License Agreement (BSD License)
#
# Copyright (c) 2012, Dorian Scholz

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want to update the copyright information?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I updated the info and changelog.


class CheckBoxGroup(QGroupBox):

def __init__(self, options, title='Checkboxes', selected_indexes=[]):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a docblock for this - especially to document what kind of information can be passed in as options.

… argument to add options directly to parent widget
@dirk-thomas
Copy link
Contributor

Squashed and merged in f4bda27.

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

Successfully merging this pull request may close these issues.

None yet

3 participants