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

jennetters fourier split 40% keymap #3487

Merged
merged 3 commits into from
Jul 26, 2018
Merged

Conversation

Jennetters
Copy link
Contributor

Add tapdance declarations for numrow and re-write keymap file for improved readability.

@drashna drashna added the keymap label Jul 25, 2018
along with this program. If not, see <http://www.gnu.org/licenses/>.
*/

#ifndef CONFIG_USER_H
Copy link
Member

Choose a reason for hiding this comment

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

Could you replace the boiler plate with #pragma once and the include here?

In fact, all you need:

#pragma once

#define TAPPING_TERM 100

/* Try to prevent sticky keys */
#define PREVENT_STUCK_MODIFIERS

/* Use I2C or Serial, not both */

#define USE_SERIAL
// #define USE_I2C

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, I overlooked this one. Added, thank you!

@@ -0,0 +1,150 @@
#include "fourier.h"
Copy link
Member

Choose a reason for hiding this comment

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

Could you replace all of the includes with just '#include QMK_KEYBOARD_H'?

@@ -0,0 +1,5 @@
ifndef QUANTUM_DIR
Copy link
Member

Choose a reason for hiding this comment

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

Could you remove this block?

You only need the tapdance line here. The rest is a hold over from the old makefile setup.

@drashna
Copy link
Member

drashna commented Jul 25, 2018

And very cool tap dances!

@drashna
Copy link
Member

drashna commented Jul 26, 2018

Awesome, thanks!

@drashna drashna merged commit e0d467d into qmk:master Jul 26, 2018
mattfluder pushed a commit to mattfluder/qmk_firmware that referenced this pull request Jul 30, 2018
* jennetters fourier keymap, tapdance declarations and reformatted keymap file for readbility

* Update rules and keymap to follow new makefile setup.

* Add #pragma once to config.h
alexey-danilov pushed a commit to alexey-danilov/qmk_firmware that referenced this pull request Aug 3, 2018
* jennetters fourier keymap, tapdance declarations and reformatted keymap file for readbility

* Update rules and keymap to follow new makefile setup.

* Add #pragma once to config.h
tenderlove added a commit to tenderlove/qmk_firmware that referenced this pull request Aug 5, 2018
* master: (184 commits)
  Docs: Clarify "right" -> "right-handed" (qmk#3507)
  Docs: Fixed the link to the advanced keycodes wiki (qmk#3505)
  Keyboard: Update KEYMAP to LAYOUT for ckeys obelus (qmk#3495)
  Keyboard: Hide JC65 directory in QMK Configurator (qmk#3491)
  Keymap: jennetters iris keymap (qmk#3488)
  Keymap: jennetters fourier split 40% keymap (qmk#3487)
  Keymap: Add LAYOUT_planck_grid to planck/dshields keymap. (qmk#3484)
  Keyboard: Add HelixPico keyboard (qmk#3502)
  Keymap: Ergodox Dvorak 42 key layout changes (qmk#3500)
  Keyboard: E6V2 Layout Fix (qmk#3490)
  Keymap: UT47:nordic - A couple of tweaks (qmk#3489)
  Keyboard: E6V2 LE Improvements (qmk#3486)
  redox: convert to use SPLIT_KEYBOARD feature (qmk#3478)
  Keymap: Bocaj Ergodox Updates (qmk#3364)
  keymap: Update khord let's split keymap and add chocopad keymap (qmk#3482)
  Keymap: Add smt keymap for TKC1800 (qmk#3481)
  Fix for the Split Common I2C code's SCL_CLOCK issue (qmk#3474)
  keymap: UT47:nordic - fixed typo (qmk#3477)
  Keyboard: Add QC60 (qmk#3472)
  Keyboard: Base for the nicekey keyboard (qmk#3475)
  ...
filoxo pushed a commit to filoxo/qmk_firmware that referenced this pull request Aug 27, 2018
* jennetters fourier keymap, tapdance declarations and reformatted keymap file for readbility

* Update rules and keymap to follow new makefile setup.

* Add #pragma once to config.h
ChrissiQ pushed a commit to ChrissiQ/qmk_firmware that referenced this pull request Sep 25, 2018
* jennetters fourier keymap, tapdance declarations and reformatted keymap file for readbility

* Update rules and keymap to follow new makefile setup.

* Add #pragma once to config.h
yamad pushed a commit to yamad/qmk_firmware that referenced this pull request Apr 10, 2019
* jennetters fourier keymap, tapdance declarations and reformatted keymap file for readbility

* Update rules and keymap to follow new makefile setup.

* Add #pragma once to config.h
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.

2 participants