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

Keymap: Adding Custom Keymap for Let's Split #9423

Merged
merged 12 commits into from
Aug 5, 2020
Merged

Keymap: Adding Custom Keymap for Let's Split #9423

merged 12 commits into from
Aug 5, 2020

Conversation

aerialviews007
Copy link
Contributor

Description

I have created a custom keymap for the Let's Split although it would also apply to other 40% keyboards as well. My objective is to emulate a dedicated numberpad. I have found that this is both easier to remember and requires the least amount of extra hand movement. With the left lower level button pressed, I can access all numbers as well as math related keys such as percentage and dollar. The function keys are then stacked on the left side which works for my needs since I do not use them much. Holding the right raise level button brings me to mouse functionality and the brackets and parentheses keys continuing to keep fingers focused on the home row.

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

@drashna drashna requested a review from a team June 18, 2020 05:29
@drashna drashna added the keymap label Jun 18, 2020
keyboards/lets_split/keymaps/aerialviews007/keymap.c Outdated Show resolved Hide resolved
keyboards/lets_split/keymaps/aerialviews007/keymap.c Outdated Show resolved Hide resolved
keyboards/lets_split/keymaps/aerialviews007/keymap.c Outdated Show resolved Hide resolved
keyboards/lets_split/keymaps/aerialviews007/keymap.c Outdated Show resolved Hide resolved
keyboards/lets_split/keymaps/aerialviews007/keymap.c Outdated Show resolved Hide resolved
keyboards/lets_split/keymaps/aerialviews007/keymap.c Outdated Show resolved Hide resolved
@stale
Copy link

stale bot commented Aug 3, 2020

Thank you for your contribution!
This pull request has been automatically marked as stale because it has not had activity in the last 45 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, or re-open when it's ready.
For maintainers: Please label with awaiting review, breaking_change, in progress, or on hold to prevent the issue from being re-flagged.

@stale stale bot removed the awaiting changes label Aug 3, 2020
@noroadsleft noroadsleft merged commit 904c561 into qmk:master Aug 5, 2020
@noroadsleft
Copy link
Member

Thanks!

@noroadsleft
Copy link
Member

For future reference, we recommend against committing to your master branch as you've done here, because pull requests from modified master branches can make it more difficult to keep your QMK fork updated. It is highly recommended for QMK development – regardless of what is being done or where – to keep your master updated, but NEVER commit to it. Instead, do all your changes in a branch (branches are basically free in Git) and issue PRs from your branches when you're developing.

There are instructions on how to keep your fork updated here:

Best Practices: Your Fork's Master: Update Often, Commit Never

Fixing Your Branch will walk you through fixing up your master branch moving forward. If you need any help with this just ask.

Thanks for contributing!

fdawans pushed a commit to fdawans/qmk_firmware that referenced this pull request Aug 11, 2020
* add

* Delete rules.mk

* Update keymap.c

* Update keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c
fdawans added a commit to fdawans/qmk_firmware that referenced this pull request Aug 11, 2020
nicocesar pushed a commit to nicocesar/qmk_firmware that referenced this pull request Aug 12, 2020
* add

* Delete rules.mk

* Update keymap.c

* Update keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c
fcoury pushed a commit to fcoury/qmk_firmware_archive that referenced this pull request Sep 20, 2020
* add

* Delete rules.mk

* Update keymap.c

* Update keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c

* Update keyboards/lets_split/keymaps/aerialviews007/keymap.c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants