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

Converting display options to checkboxes - WIP #8

Merged
merged 1 commit into from
Aug 6, 2019
Merged

Conversation

thundernixon
Copy link
Collaborator

I'm trying to change the "Display" options from a segmented control to individual checkboxes.

I've found that the current UI control is using a vanilla SegmentedButton, and this is sending a number to a callback function which sets display options based on the segmented selection. So, I think I need to make three individual vanilla checkBoxes, and let them call the same function, but send a title or integer when their value is True.

However, I'm currently stuck. Even when simply trying to make a single checkbox and print it's sender value from the callback, I get the following error message:

********************
Installing 'mirroringPane' report:
  Traceback (most recent call last):
      File "mirroredBackground.py", line 353, in <module>
      File "mirroredBackground.py", line 36, in __init__
      File "mirroredBackground.py", line 71, in initUI
      File "/Applications/RoboFont3.3b.app/Contents/Resources/lib/python3.6/vanilla/vanillaCheckBox.py", line 75, in __init__
      File "/Applications/RoboFont3.3b.app/Contents/Resources/lib/python3.6/vanilla/vanillaCheckBox.py", line 83, in set
    ValueError: depythonifying 'long long', got 'str' of 5
  
********************

I'm not quite sure where to go from here, except ... to bed, for now. I'll pick this up again later. :) In the meantime, I would love to get a pointer as to what I might be missing, here.

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