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 ortho_4x12 & planck_mit layouts for KBD4X. #5729

Merged
merged 3 commits into from
Apr 30, 2019

Conversation

newtonapple
Copy link
Contributor

Enabling ortho_4x12 & planck_mit layouts for KBD4X.

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

Copy link
Member

@noroadsleft noroadsleft left a comment

Choose a reason for hiding this comment

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

Travis CI error is because bakingpy's (@nooges) ortho_4x12 keymap uses LAYOUT_kc_ortho_4x12 for its macro, which isn't supported here.

I don't think he owns a KBD4x, so I'm going to call this good.

{ k30, k31, k32, k33, k34, k35, k36, k37, k38, k39, k3a, k3b } \
}

#define LAYOUT_kc_ortho_4x12( \
Copy link
Member

Choose a reason for hiding this comment

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

I believe the use of layout macros which automatically prepend KC_ is discouraged - it makes it difficult to use keycodes like MO() etc.

Copy link
Member

Choose a reason for hiding this comment

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

I like it though, so it should probably stay.

Copy link
Member

Choose a reason for hiding this comment

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

I like it though, so it should probably stay.

:(

@newtonapple
Copy link
Contributor Author

I added the macro and now the firmware is too large haha... So, there are couple things I usually do. On my personal keymaps, I usually add the following to the config.h

#define NO_ACTION_MACRO
#define NO_ACTION_FUNCTION

And add EXTRAFLAGS += -flto to the rules.mk.

Also, but the KBD4x also doesn't come w/ backlight. So, turning that off would save a lot of space. I can do that if you guys are ok with it.

@nooges
Copy link
Member

nooges commented Apr 29, 2019

Yeah, good idea to turn backlight off in rules.mk then.

@noroadsleft
Copy link
Member

I'd actually prefer if Backlight stayed on.

Switching CONSOLE_ENABLE to no should save close to 6kB.

@drashna drashna added the keymap label Apr 30, 2019
@drashna drashna added keyboard and removed keymap labels Apr 30, 2019
@drashna drashna merged commit e77e46f into qmk:master Apr 30, 2019
akrob pushed a commit to akrob/qmk_firmware that referenced this pull request Apr 30, 2019
* upstream/master: (779 commits)
  [Keyboard] Signum3.0 remove sortedcontainers (qmk#5679)
  Simple extended space cadet (qmk#5277)
  Removed forced in lining for lib8tion functions (qmk#5670)
  Change lib8tion library to be usable in user keymaps (qmk#5598)
  [Keyboard] Fixing drag-and-drop (qmk#5728)
  [Keyboard] Adding ortho_4x12 & planck_mit layouts for KBD4X (qmk#5729)
  [Keyboard] Minor fixes for Baguette (qmk#5737)
  Updated rgb_led struct field modifier to flags (qmk#5619)
  RGB Matrix: Custom effects on a kb/user level (qmk#5338)
  Fix Planck and Preonic builds (qmk#5658)
  [Keymap] dz60 keymap w/ hhkb-esque default layer (qmk#5708)
  [Keymap] Added compatibility for Planck rev6 (qmk#5706)
  [Keyboard] Satisfaction75 i2c fix and VIA layout (qmk#5726)
  A better new_project.sh (qmk#5191)
  Fix sendstring "#" producing "£" instead (qmk#5724)
  [Keyboard] Added WT69-A PCB (qmk#5721)
  [Keymap] Fix typo and function layer image for Quefrency (qmk#5719)
  [Keymap] Initial keyboard layout for KBD67 (qmk#5720)
  [Keymap] New keymap for Quefrency 65% with split backspace, RGB, media keys, mouse keys (qmk#5717)
  [Keyboard] Update Gergo to use newer Ergodox Matrix code (qmk#5703)
  ...
foosinn pushed a commit to foosinn/qmk_firmware that referenced this pull request May 6, 2019
* Adding ortho_4x12 & planck_mit layouts for KBD4X.

* Adding LAYOUT_kc_ortho_4x12 macro to KBD4x.

* Turn off console for KBD4X so firmware size falls within limit.
Timbus pushed a commit to Timbus/qmk_firmware that referenced this pull request Jun 23, 2019
* Adding ortho_4x12 & planck_mit layouts for KBD4X.

* Adding LAYOUT_kc_ortho_4x12 macro to KBD4x.

* Turn off console for KBD4X so firmware size falls within limit.
@newtonapple newtonapple deleted the kbd4x branch August 31, 2019 04:59
ridingqwerty pushed a commit to ridingqwerty/qmk_firmware that referenced this pull request Jan 10, 2020
* Adding ortho_4x12 & planck_mit layouts for KBD4X.

* Adding LAYOUT_kc_ortho_4x12 macro to KBD4x.

* Turn off console for KBD4X so firmware size falls within limit.
JeffreyPalmer pushed a commit to JeffreyPalmer/qmk_firmware that referenced this pull request Feb 27, 2020
* Adding ortho_4x12 & planck_mit layouts for KBD4X.

* Adding LAYOUT_kc_ortho_4x12 macro to KBD4x.

* Turn off console for KBD4X so firmware size falls within limit.
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.

5 participants