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

Place newly created modules directly under the cursor #28

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

samuellwn
Copy link

This PR calculates a better position for new modules. The old method could be badly wrong, especially in multi-monitor setups. I only tested on Linux.

@samuellwn
Copy link
Author

This doesn't account for zoom. If the graph has been zoomed in or out the position will be inaccurate.

@drobilla
Copy link
Owner

Thanks, I'll check it out. I had noticed that the module position is wonky sometimes but hadn't looked into it (largely because I'm in the process of rewriting the whole canvas anyway, but that will still be a while...)

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