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

[Pointing] Add uart based SpaceMouse Module support #22519

Draft
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

drashna
Copy link
Member

@drashna drashna commented Nov 21, 2023

Description

Title.

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Copy link
Contributor

@keyboard-magpie keyboard-magpie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this looks good. Obviously no way at present to map to 3dconnexions 3d axis system. Need to have a look and see how the orbion handles things.

@drashna
Copy link
Member Author

drashna commented Nov 21, 2023

I think this looks good. Obviously no way at present to map to 3dconnexions 3d axis system. Need to have a look and see how the orbion handles things.

it's 3 axis, so it's not really a good comparison.

But it wouldn't be hard to implement scrolling, both horizontal and vertical with the tilt or shift axis, and use the other for mouse movement. but would have to check both sets and filter out, since it is very easy to trigger both (ran into that in testing).

However, I was mostly concerned with getting it working. And there is a provided way to customize how the reporting is translated, which I think is good enough for now.

quantum/pointing_device/pointing_device_drivers.c Outdated Show resolved Hide resolved
docs/feature_pointing_device.md Outdated Show resolved Hide resolved
quantum/pointing_device/pointing_device_drivers.c Outdated Show resolved Hide resolved
Comment on lines +529 to +530
.set_cpi = NULL,
.get_cpi = NULL
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It'd be nice to have some form of software scaling here.

drivers/sensors/spacemouse_module.h Outdated Show resolved Hide resolved
drivers/sensors/spacemouse_module.c Outdated Show resolved Hide resolved
drivers/sensors/spacemouse_module.c Outdated Show resolved Hide resolved
@drashna drashna force-pushed the space_mouse_coast_to_coast branch 3 times, most recently from 5587275 to 2c02433 Compare November 29, 2023 11:43
@drashna drashna marked this pull request as draft January 7, 2024 07:42
drashna and others added 10 commits June 9, 2024 17:32
To be honest, there isn't a lot that needs to be customized here, but there is a lot that could/shold be done for handling the other axises.  But drashna doesn't want to do it, at this time
Re-fix lint?!
Co-authored-by: Dasky <32983009+daskygit@users.noreply.github.com>
@drashna drashna force-pushed the space_mouse_coast_to_coast branch from 63cc65f to ca6caeb Compare June 10, 2024 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants