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

Port 'Box' demo to Python #76

Merged
merged 1 commit into from
Feb 11, 2024
Merged

Conversation

rolandlo
Copy link
Contributor

@rolandlo rolandlo commented Feb 9, 2024

I followed the Javascript example closely. One difference lies in the append function, where the signal handler must take the button as a function parameter, but the call to append() at the end doesn't pass one. That's why I used default parameter None.
The code appears to work fine.

@rolandlo rolandlo changed the title Port 'Grid' demo to Python Port 'Box' demo to Python Feb 9, 2024
Copy link
Contributor

@theCapypara theCapypara left a comment

Choose a reason for hiding this comment

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

Thanks! There are a few things here I would like to be less verbose, but this would require restructuring some parts of the demo across all languages + a bit of verbosity isn't bad for demo code I guess.

@theCapypara theCapypara merged commit 6495014 into workbenchdev:main Feb 11, 2024
1 check passed
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