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: implements mouse control support for MacOS #1035

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Kircheneer
Copy link
Contributor

Describe your changes. Use imperative present tense.

Implemented mouse control support for MacOS.

Checklist

  • Add documentation to docs/config.adoc
    • N/A - already in
    • Updated known issues for platforms
  • Add example and basic docs to cfg_samples/kanata.kbd
    • N/A - already in
  • Update error messages
    • Yes
  • Added tests, or did manual testing
    • Yes

@Kircheneer Kircheneer force-pushed the macos-mouse-control branch 2 times, most recently from 93ae887 to 2da27c2 Compare May 14, 2024 08:51
@nightscape
Copy link
Contributor

I just tried this, great job!
Mouse movement and mrgt work without issues, but mlft does not click.
@Kircheneer same for you?

@Kircheneer
Copy link
Contributor Author

I plan to address @nightscape's comments, just haven't had a lot of time to check yet.

@Kircheneer
Copy link
Contributor Author

I have tried with the following config:

(defsrc
e
)

(defalias
e mlft
)

(deflayer default
@e
)

and this worked flawlessly. Can you share what config you are on maybe?

@nightscape
Copy link
Contributor

nightscape commented May 28, 2024

@Kircheneer I just tried again with your configuration and I'm still observing the same behaviour:
I can click on certain things, e.g. the Mac OS Menu Bar, but not others, e.g. to change a tab in iTerm.
Do you have iTerm installed? If so, could you try switching tabs using the mlft?

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

2 participants