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

VIA Support & LAYOUT macro fixing: KBDFans KBD19x #10401

Merged
merged 13 commits into from
Oct 1, 2020

Conversation

mechmerlin
Copy link
Contributor

@mechmerlin mechmerlin commented Sep 23, 2020

Description

Added a VIA keymap and had to also edit the kbd19x.h file as the matrix key labeling was completely ALL OVER THE PLACE.

Types of Changes

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

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

@mechmerlin mechmerlin added the via Adds via keymap and/or updates keyboard for via support label Sep 23, 2020
@mechmerlin mechmerlin changed the title Feature/kbd19x via VIA Support: KBDFans KBD19x Sep 23, 2020
@mechmerlin mechmerlin changed the title VIA Support: KBDFans KBD19x VIA Support & LAYOUT macro fixing: KBDFans KBD19x Sep 27, 2020
@fauxpark fauxpark requested a review from a team October 1, 2020 18:39
@noroadsleft noroadsleft merged commit b31df1a into qmk:master Oct 1, 2020
@noroadsleft
Copy link
Member

Thanks!

rgoulter pushed a commit to rgoulter/qmk_firmware that referenced this pull request Oct 4, 2020
* first pass at fixing the key labeling

* second pass of fixing key labeling

* third pass of this key naming

* another pass of changing key naming

* another pass

* another pass

* another pass

* renamed it all

* give a good vid and pid

* add via keymap

* add on to the comments

* update readme and comments

* one more small change
kjganz pushed a commit to kjganz/qmk_firmware that referenced this pull request Oct 28, 2020
* first pass at fixing the key labeling

* second pass of fixing key labeling

* third pass of this key naming

* another pass of changing key naming

* another pass

* another pass

* another pass

* renamed it all

* give a good vid and pid

* add via keymap

* add on to the comments

* update readme and comments

* one more small change
@mechmerlin mechmerlin deleted the feature/kbd19x_via branch January 26, 2021 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keyboard keymap via Adds via keymap and/or updates keyboard for via support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants