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

fix: add support for mac's command key for selection manager #3315

Conversation

rein1410
Copy link
Contributor

@rein1410 rein1410 commented Jan 13, 2025

Description

Selection manager was missing support for multiple selection with the mac's command key, causing Mac users unable to select images one by one. since holding control on Mac opens the right click menu on chrome based browsers.

Breaking changes

No

Checklist

📌 Always:

  • I have set a clear title
  • My PR is small and contains a single feature
  • I have checked my own PR

👍 Most of the time:

  • I have added or updated test cases
  • I have updated the README if needed

Copy link

vercel bot commented Jan 13, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview Jan 13, 2025 8:01am

Copy link
Contributor

github-actions bot commented Jan 13, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@rein1410 rein1410 changed the title fix: add support for command key for selection manager fix: add support for mac's control key for selection manager Jan 13, 2025
@rein1410 rein1410 changed the title fix: add support for mac's control key for selection manager fix: add support for mac's command key for selection manager Jan 13, 2025
@michaelbromley
Copy link
Member

michaelbromley commented Jan 21, 2025

Thanks for the contribution! This looks good. Before I can merge this, you'll need to sign the CLA as directed in this comment: #3315 (comment)

And also can you make the PR against the master branch? This can be counted as a fix IMO.

@rein1410 rein1410 changed the base branch from minor to master January 22, 2025 01:21
@rein1410
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Jan 22, 2025
@michaelbromley michaelbromley merged commit c1cfb73 into vendure-ecommerce:master Jan 22, 2025
31 checks passed
@michaelbromley
Copy link
Member

Thank you!

@github-actions github-actions bot locked and limited conversation to collaborators Jan 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants