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: recognize command+click on macos #2786

Conversation

eshack94
Copy link
Contributor

@eshack94 eshack94 commented May 19, 2024

Summary

Fixes an issue where "command+click" combination was not being recognized on MacOS. The desired behavior was working fine on Windows using "ctrl+click", but the MacOS equivalent was broken.

This was preventing new tabs from opening while holding "command" (meta key) on MacOS and clicking.

Change Type

  • Bug fix (non-breaking change which fixes an issue

Testing

I verified this change fixes the issue by building locally and testing.

After making changes locally, I re-built the docker images locally, ran the app, and verified the issue is fixed.

Test Configuration:

Docker Compose, after making changes and rebuilding (without cache) locally.

Checklist

Please delete any irrelevant options.

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • My changes do not introduce new warnings
  • Local unit tests pass with my changes - WIP

Fixes an issue where the "command+click" was not being recognized on
MacOS. The desired behavior was working fine on Windows using
"ctrl+click", but the MacOS equivalent was broken.

This was preventing new tabs from opening while holding "command" (meta
key) on MacOS and clicking.

I verified this change fixes the issue by building locally and testing.
@eshack94
Copy link
Contributor Author

This addresses #2784.

@danny-avila danny-avila merged commit af8bcb0 into danny-avila:main May 19, 2024
techwithanirudh pushed a commit to techwithanirudh/LibreChat-dev that referenced this pull request May 20, 2024
Fixes an issue where the "command+click" was not being recognized on
MacOS. The desired behavior was working fine on Windows using
"ctrl+click", but the MacOS equivalent was broken.

This was preventing new tabs from opening while holding "command" (meta
key) on MacOS and clicking.

I verified this change fixes the issue by building locally and testing.
danny-avila pushed a commit that referenced this pull request Aug 5, 2024
Fixes an issue where the "command+click" was not being recognized on
MacOS. The desired behavior was working fine on Windows using
"ctrl+click", but the MacOS equivalent was broken.

This was preventing new tabs from opening while holding "command" (meta
key) on MacOS and clicking.

I verified this change fixes the issue by building locally and testing.
kenshinsamue pushed a commit to intelequia/LibreChat that referenced this pull request Sep 17, 2024
Fixes an issue where the "command+click" was not being recognized on
MacOS. The desired behavior was working fine on Windows using
"ctrl+click", but the MacOS equivalent was broken.

This was preventing new tabs from opening while holding "command" (meta
key) on MacOS and clicking.

I verified this change fixes the issue by building locally and testing.
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