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

Added Space Cadet modifer carry over #5866

Merged
merged 1 commit into from
Jul 7, 2019

Conversation

XScorpion2
Copy link
Contributor

Description

This is an option for those fast typing folks that carry over the mods from when you pressed the space cadet key to the release. For example: Press Shift > Press KC_LSPO > Release Shift > Release KC_LSPO. In this example, since KC_LSPO tap triggers on release, it will not contain the Shift modifier from when it was pressed. This define will carry over the modifiers from when it was pressed and use them on tap trigger release. This is super useful when using a setup like this and fast typing to get both { & [ based on if the other shift is held down or not.

#define LSPO_KEYS KC_LSFT, KC_TRNS, KC_LBRC
#define RSPC_KEYS KC_RSFT, KC_TRNS, KC_RBRC

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.
  • 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).

@XScorpion2 XScorpion2 changed the title Added mod carry over from press to release. Added Space Cadet modifer carry over May 14, 2019
@XScorpion2 XScorpion2 force-pushed the features/space_cadet_carry branch from f8fcce5 to dc43b45 Compare May 20, 2019 00:24
@XScorpion2 XScorpion2 force-pushed the features/space_cadet_carry branch from dc43b45 to 6dcfc84 Compare June 3, 2019 00:17
Update docs/feature_space_cadet.md

Co-Authored-By: fauxpark <fauxpark@gmail.com>
@XScorpion2 XScorpion2 force-pushed the features/space_cadet_carry branch from 6dcfc84 to db44a9f Compare June 30, 2019 18:34
@XScorpion2
Copy link
Contributor Author

Rebased on latest master

Copy link
Contributor

@mechmerlin mechmerlin left a comment

Choose a reason for hiding this comment

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

Thanks!

@mechmerlin mechmerlin merged commit d16db69 into qmk:master Jul 7, 2019
@XScorpion2 XScorpion2 deleted the features/space_cadet_carry branch July 11, 2019 23:16
drashna pushed a commit to zsa/qmk_firmware that referenced this pull request Jul 12, 2019
Co-Authored-By: fauxpark <fauxpark@gmail.com>
fdidron pushed a commit to zsa/qmk_firmware that referenced this pull request Jul 27, 2019
Co-Authored-By: fauxpark <fauxpark@gmail.com>
doughsay pushed a commit to doughsay/qmk_firmware that referenced this pull request Aug 31, 2019
Update docs/feature_space_cadet.md

Co-Authored-By: fauxpark <fauxpark@gmail.com>
ripxorip pushed a commit to ripxorip/qmk_firmware that referenced this pull request Dec 3, 2019
Update docs/feature_space_cadet.md

Co-Authored-By: fauxpark <fauxpark@gmail.com>
ridingqwerty pushed a commit to ridingqwerty/qmk_firmware that referenced this pull request Jan 10, 2020
Update docs/feature_space_cadet.md

Co-Authored-By: fauxpark <fauxpark@gmail.com>
JeffreyPalmer pushed a commit to JeffreyPalmer/qmk_firmware that referenced this pull request Feb 27, 2020
Update docs/feature_space_cadet.md

Co-Authored-By: fauxpark <fauxpark@gmail.com>
swamp09 pushed a commit to swamp09/qmk_firmware that referenced this pull request Mar 11, 2020
Update docs/feature_space_cadet.md

Co-Authored-By: fauxpark <fauxpark@gmail.com>
BorisTestov pushed a commit to BorisTestov/qmk_firmware that referenced this pull request May 23, 2024
Update docs/feature_space_cadet.md

Co-Authored-By: fauxpark <fauxpark@gmail.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.

4 participants