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

Adding Navi10 macropad #7556

Merged
merged 16 commits into from
Dec 9, 2019
Merged

Adding Navi10 macropad #7556

merged 16 commits into from
Dec 9, 2019

Conversation

emdarcher
Copy link
Contributor

Adding the Navi10 macropad to QMK.

Description

Adding my new macropad, the Navi10, to QMK.

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

@noroadsleft noroadsleft requested a review from a team December 7, 2019 06:44
keyboards/navi10/keymaps/default/config.h Outdated Show resolved Hide resolved
keyboards/navi10/keymaps/emdarcher/config.h Outdated Show resolved Hide resolved
keyboards/navi10/readme.md Outdated Show resolved Hide resolved
keyboards/navi10/readme.md Outdated Show resolved Hide resolved
keyboards/navi10/readme.md Outdated Show resolved Hide resolved
keyboards/navi10/rev0/rules.mk Outdated Show resolved Hide resolved
keyboards/navi10/rev0/rules.mk Outdated Show resolved Hide resolved
keyboards/navi10/rev0/rules.mk Outdated Show resolved Hide resolved
keyboards/navi10/rev2/rules.mk Outdated Show resolved Hide resolved
keyboards/navi10/rev3/rules.mk Outdated Show resolved Hide resolved
keyboards/navi10/readme.md Outdated Show resolved Hide resolved
keyboards/navi10/readme.md Outdated Show resolved Hide resolved
Copy link
Member

@drashna drashna left a comment

Choose a reason for hiding this comment

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

I'm not a fan of having tapdance enabled in the default keymap. Even less so having it enabled "globally" via the keyboard's rules.mk file.

I'm not sure what the best solution would be, long term, but for now, could you move the TAP_DANCE_ENABLE = yes to a rules.mk file in the default keymap folder?

@noroadsleft noroadsleft merged commit 6c834de into qmk:master Dec 9, 2019
@noroadsleft
Copy link
Member

Thanks!

@emdarcher emdarcher deleted the navi10_branch branch December 9, 2019 06:43
@emdarcher emdarcher restored the navi10_branch branch December 9, 2019 18:57
patrl pushed a commit to patrl/qmk_firmware that referenced this pull request Dec 29, 2019
* working on keymap and files for the navi10

* fixing and modifying tap dance keys and layers

* cleanup, and adding my separate keymap

* edited and simplified the default keymap

* cleaned up files and added support for future rev2 board

* readme edits

* moved rev0 specific keymap to it's own folder

* added revision data to the config files

* added DEFAULT_FOLDER

* added note on compiling for a specific revision

* documentation and readme edits

* moved keymaps around and cleaned up

* added photo

* formatting and cleanup

* fixing whitespace

* moving the tap dance enable to keymap folders
calmh added a commit to calmh/qmk_firmware that referenced this pull request Jan 17, 2020
* upstream/master: (1960 commits)
  [Keyboard] Think6.5 Default Keymap Cleanup (qmk#7594)
  [Keyboard] update m12og keymap to readable version (qmk#7581)
  [Keyboard] Add cKeys Washington keyboard (qmk#7570)
  [Keyboard] Fix 2U backspace key in ansi_blocker layout for GrayStudio Space65 keyboard (qmk#7593)
  [Keyboard] Remove `PREVENT_STUCK_MODIFIERS` from config.h files (qmk#7592)
  [Keyboard] update default h87a keymap for fn functionality (qmk#7589)
  [Keymap] ergotravel updates (qmk#7588)
  [Keyboard] Adjust ColorLice to work with QMK configurator (qmk#7572)
  [Keyboard] Adding Navi10 macropad (qmk#7556)
  Enable bitbang ws2812 for f4 (qmk#7571)
  [Keyboard] add Matrix 8XV1.2 og ISO/ANSI keyboard (qmk#7567)
  Also fix flash
  Fix compiling json files
  Add JSON keymap to prereq list of C keymap
  [Keyboard] instant60: Enable bootmagic lite to make eeprom reset easier (qmk#7566)
  Updated slave encoder sync to reduce dropped pulses - v2 (qmk#7505)
  [Keyboard] rebuild info.json for ep tf_longeboye (qmk#7560)
  [Keyboard] Added workaround for serial.c/LTO issue for helix/rev2 and helix/pico. (qmk#7558)
  [Keyboard] EP96: fix info.json key sequence (qmk#7557)
  [Keymap] Added my customized 40% layout (qmk#7555)
  ...
HokieGeek pushed a commit to HokieGeek/qmk_firmware that referenced this pull request Feb 21, 2020
* working on keymap and files for the navi10

* fixing and modifying tap dance keys and layers

* cleanup, and adding my separate keymap

* edited and simplified the default keymap

* cleaned up files and added support for future rev2 board

* readme edits

* moved rev0 specific keymap to it's own folder

* added revision data to the config files

* added DEFAULT_FOLDER

* added note on compiling for a specific revision

* documentation and readme edits

* moved keymaps around and cleaned up

* added photo

* formatting and cleanup

* fixing whitespace

* moving the tap dance enable to keymap folders
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.

None yet

4 participants