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

Usability Fixes #243

Merged
merged 4 commits into from
Jun 8, 2022
Merged

Usability Fixes #243

merged 4 commits into from
Jun 8, 2022

Conversation

aaronvark
Copy link
Contributor

Implemented multi-select copy/paste. This basically takes the title, zconfig capabilities string & relative position of any selected actor, and uses CreateFromType and a new function "SetCapabilities" to re-create & inject the copied actor. Also respects max actor count (on paste)

Removed the enter restriction from input fields. Actors should be able to deal with nonsensical input (since they do this even when we hit enter). There is an ImGui issue that's been open for years that seeks to address this, maybe we can help it along to make this better: ocornut/imgui#701

Issue: we're hitting socket limits if we copy/paste a lot of nodes, so we should probably handle this "exceeded max actors" more gracefully. Or just increase the socket limit through the environment to some ridiculous amount (or at this point).

Also includes some fixes:

  • DmxActor wasn't returning the list of port names on OSX
  • NatNetActor would hang if timeout was set to 0 (so minimum 1 was added)

@sphaero sphaero merged commit 69e0f2d into hku-ect:master Jun 8, 2022
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