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: add Meta key support for accumulation function #22

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

MdSajjadurRahaman
Copy link
Contributor

@MdSajjadurRahaman MdSajjadurRahaman commented Aug 20, 2024

The accumulation function now recognizes Meta key presses in addition to the Control key. This change enhances cross-platform compatibility, particularly for Mac users who use the Meta key for similar operations.

Description

This update modifies the accumulation function to detect Meta key presses, in addition to the existing Control key support. The primary motivation behind this change is to improve the user experience for Mac users, where the Meta key (often the Command key) is frequently used for operations similar to those performed with the Control key on other platforms.

Checklist

The accumulation function now recognizes `Meta` key presses in addition to the `Control` key. This change enhances cross-platform compatibility, particularly for Mac users who use the `Meta` key for similar operations.
@MdSajjadurRahaman
Copy link
Contributor Author

@Ni55aN Could you please look into this? Alternatively, is there another way to handle support for the 'COMMAND' key when selecting multiple nodes?

@Ni55aN Ni55aN merged commit 70774f1 into retejs:main Aug 29, 2024
2 of 4 checks passed
@rete-js
Copy link
Contributor

rete-js bot commented Aug 29, 2024

🎉 This issue has been resolved in version 2.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@rete-js rete-js bot added the released label Aug 29, 2024
@Ni55aN
Copy link
Member

Ni55aN commented Aug 29, 2024

@MdSajjadurRahaman done

@MdSajjadurRahaman
Copy link
Contributor Author

Thank you🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants