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

feat(editor): Improve editor control #372

Closed
wants to merge 1 commit into from

Conversation

starker-xp
Copy link
Contributor

@starker-xp starker-xp commented Feb 1, 2024

Added the ability to select multiple nodes at the same time.
Allows you to deselect an already selected node.

Caution: Changes the way you move with the mouse. You'll need to use the Control key to move the pan.

Based on this demand :
#368, #298

Example :
image

…time. Allows you to deselect an already selected node.

Caution: Changes the way you move with the mouse. You'll need to use the Control key to move the pan.
@yojeek
Copy link
Contributor

yojeek commented Feb 8, 2024

Appreciated feature. However I feel that panning is more often used than selection of multiple items, thus maybe it's better to pan without extra key modifiers.

@newcat
Copy link
Owner

newcat commented Feb 8, 2024

That's also the reason I didn't merge it. Because I also noticed a few other points, I'll take your implementation as a starting point and implement it myself, but thanks for the initial implementation and the PR!

newcat added a commit that referenced this pull request Feb 8, 2024
@newcat newcat closed this Feb 8, 2024
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.

None yet

3 participants