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

Add new keyboard "Left Slinger" #24618

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

takashicompany
Copy link
Contributor

@takashicompany takashicompany commented Nov 19, 2024

Description

I have designed a new keyboard and would appreciate a review.

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout (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).

@drashna drashna requested a review from a team December 2, 2024 03:59
@takashicompany
Copy link
Contributor Author

@drashna Can you please review this PR?

keyboards/takashicompany/left_slinger/config.h Outdated Show resolved Hide resolved
keyboards/takashicompany/left_slinger/custom_joystick.c Outdated Show resolved Hide resolved
keyboards/takashicompany/left_slinger/keyboard.json Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

Is there a specific reason for this custom code over using the core analog joystick driver?

This is simpler, yes, but loses out on some of the improvements that the core has.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://x.com/takashicompany/status/1742946762447573352

The joystick on this keyboard was incompatible with the QMK Firmware joystick implementation, and the pointer movement was linear in response to stick input. My implementation is at least compatible with the joystick that will be installed, so I wrote the source code separately.

}

// クリック用のレイヤーを有効にする。 Enable layers for clicks
void enable_click_layer(void) {
Copy link
Member

Choose a reason for hiding this comment

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

This seems to be a re-implementation of the pointing device auto mouse layer feature.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The implementation is certainly close to Automatic Mouse Layer. However, I would like to use this source code if it is not a problem, as I would like to make modifications suitable for keyboards in the future. Please consider this.

keyboards/takashicompany/left_slinger/left_slinger.c Outdated Show resolved Hide resolved
takashicompany and others added 6 commits January 14, 2025 21:19
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
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.

2 participants