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

Move kwerdenker's personal keymap from RGB #7645

Merged
merged 6 commits into from
Dec 16, 2019

Conversation

yanfali
Copy link
Contributor

@yanfali yanfali commented Dec 15, 2019

Talked to Maartenwut about making RGB an actual template instead of
kwerdenker's keymap. Free'd up B7 so it can be used for audio in
agreement with MW. He's considering adding a speaker footprint to next
rev of plain60, and only B7 can be used with Audio. RGB uses bitbang
driver so it doesn't really make any difference if it's pin B0.

  • RGB should be a template for other people to modify, not a personal
    keymap
  • change default pin to B0 to free up B7 for audio use
  • rename RGB to kwerdenker

@kwerdenker you ok with this? Basically it's to add a new feature to plain60 pcb, and I noticed that the RGB keymap was not generic but highly specific. I've based new RGB keymap on default.

Description

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

Yan-Fa Li added 6 commits December 15, 2019 08:39
Talked to Maartenwut about making RGB an actual template instead of
kwerdenker's keymap. Free'd up B7 so it can be used for audio in
agreement with MW. He's considering adding a speaker footprint to next
rev of plain60, and only B7 can be used with Audio. RGB uses bitbang
driver so it doesn't really make any difference if it's pin B0.

 - RGB should be a template for other people to modify, not a personal
 keymap
 - change default pin to B0 to free up B7 for audio use
 - rename RGB to kwerdenker
 - doesn't work properly on MacOS or Windows
 - maintain compatibility with old keymap
@drashna drashna requested a review from a team December 16, 2019 07:07
@drashna drashna added the keymap label Dec 16, 2019
@kwerdenker
Copy link
Contributor

Don't have a problem with it. Thought nobody beside myself did any mods to the PCB, so saw no need to generalize the keymap. Thank you for the work @yanfali

@drashna drashna merged commit 0082ecf into qmk:master Dec 16, 2019
@yanfali yanfali deleted the yanfali-plain60-kwerdenker branch December 17, 2019 17:10
benjaminmikiten added a commit to benjaminmikiten/qmk_firmware that referenced this pull request Dec 18, 2019
* master: (99 commits)
  [Keymap] Added userspace for d4mation. Included their keymap for the Atreus62 (qmk#7483)
  [Keymap] Custom user keymap for Think6.5 with LED range control (qmk#7603)
  [Keymap] CRKBD Custom Keymap - KidBrazil (qmk#7630)
  [Keymap] Add pico 70 keys keymap (qmk#7654)
  Tidy up dztech default keymaps and info.json (qmk#7608)
  Heisenberg handwired keyboard added (qmk#7643)
  [Keyboard] Added Filco Majestouch TKL Pegasus Hoof ISO Layout (qmk#7647)
  Ported J80 to QMK (qmk#7488)
  [Keyboard] Magnavox Videowriter conversion with Pro Micro (qmk#7634)
  [Docs] add japanese translation (basic part) (qmk#7461)
  Tidy up dztech rules.mk
  Relocate RGB keycode processing (qmk#7508)
  Move kwerdenker's personal keymap from RGB (qmk#7645)
  Remove QMK_KEYBOARD_CONFIG_H from boards (qmk#7635)
  Disable usb on slave half to resolve random 'lockup' (qmk#7649)
  [Core] Optimize matrix processing (qmk#7621)
  [Keymap] boy_314's satisfaction75 layout (qmk#7638)
  [Keyboard] XD68 65% ATMega32U4 based (qmk#7395)
  [keyboard] Plain60 cleanups (qmk#7644)
  update default h88 keymap (qmk#7646)
  ...
patrl pushed a commit to patrl/qmk_firmware that referenced this pull request Dec 29, 2019
* Move kwerdenker's personal keymap from RGB

Talked to Maartenwut about making RGB an actual template instead of
kwerdenker's keymap. Free'd up B7 so it can be used for audio in
agreement with MW. He's considering adding a speaker footprint to next
rev of plain60, and only B7 can be used with Audio. RGB uses bitbang
driver so it doesn't really make any difference if it's pin B0.

 - RGB should be a template for other people to modify, not a personal
 keymap
 - change default pin to B0 to free up B7 for audio use
 - rename RGB to kwerdenker

* Add RGB keycodes to FN layer

* don't use uppercase in keymap names

 - doesn't work properly on MacOS or Windows

* Add an audio specific keymap

* Enum instead of define

* Suggested change to keymap

 - maintain compatibility with old keymap
HokieGeek pushed a commit to HokieGeek/qmk_firmware that referenced this pull request Feb 21, 2020
* Move kwerdenker's personal keymap from RGB

Talked to Maartenwut about making RGB an actual template instead of
kwerdenker's keymap. Free'd up B7 so it can be used for audio in
agreement with MW. He's considering adding a speaker footprint to next
rev of plain60, and only B7 can be used with Audio. RGB uses bitbang
driver so it doesn't really make any difference if it's pin B0.

 - RGB should be a template for other people to modify, not a personal
 keymap
 - change default pin to B0 to free up B7 for audio use
 - rename RGB to kwerdenker

* Add RGB keycodes to FN layer

* don't use uppercase in keymap names

 - doesn't work properly on MacOS or Windows

* Add an audio specific keymap

* Enum instead of define

* Suggested change to keymap

 - maintain compatibility with old keymap
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