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

Choice button: Use proper enum index even if options differ from VEnum #2630

Merged
merged 1 commit into from
Apr 10, 2023

Conversation

kasemir
Copy link
Collaborator

@kasemir kasemir commented Apr 10, 2023

Allow choice buttons with custom options A, C, E for PVs with enum labels A, B, C, D, E

#2629

Without "Items from PV" selected, the PV's value may not match any of the custom choice button options.
Indicate with same border as "disconnected". This may be an overload of the border meaning, but avoids having to invent another way to indicate the issue. The border is quite obvious in indicating a problem, the tool tip will show the current value and help explain why no button option is selected, and in a way the value of the PV is disconnected from the available choices.

@tanviash
Copy link
Collaborator

@kasemir Thank you for the fix.

@tanviash tanviash merged commit bcfad51 into master Apr 10, 2023
@tanviash tanviash deleted the choice_button_options branch April 10, 2023 17:52
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.

2 participants