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

Personal Nyquist keymap added. #4059

Merged
merged 3 commits into from
Oct 7, 2018
Merged

Personal Nyquist keymap added. #4059

merged 3 commits into from
Oct 7, 2018

Conversation

BramVer
Copy link
Contributor

@BramVer BramVer commented Oct 3, 2018

I hope I get the conventions right from the first time now.

Nothing fancy, just a keymap which works for me, always great to share with the rest.

keyboards/nyquist/keymaps/bramver/keymap.c Outdated Show resolved Hide resolved

void matrix_init_user(void) {
_delay_ms(20); // Gets rid of tick
set_unicode_input_mode(UC_LNX);
Copy link
Member

Choose a reason for hiding this comment

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

You should only need to set this once. Once it's set, you're good to go.

Also the delay isn't actually needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What do you mean 'only set it once'? Should this be defined somewhere else/in a different fashion?

Copy link
Member

Choose a reason for hiding this comment

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

Nope. This function actually writes to EEPROM, so it should be loaded automatically on future loads. In theory.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay, so what I got from this, was:

  • get rid of the matrix_init_user() function since it's already set and doing this would set it twice
  • use UC_LNX(unicode_map_index) to send unicode strings up to 0xFFFF (which the defined emojis fall under)

Am I correct in this, or just completely missing what you're saying? :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So the above did not work since it wouldn't compile.
I just removed the delay like you said, but no clue how I can correctly input these unicodes otherwise, so no real changes there.
If I'm missing something from the docs, please do explain because I'm not seeing it :)

keyboards/nyquist/keymaps/bramver/keymap.c Outdated Show resolved Hide resolved
Removed unnecessary imports.
Updated way of macros.
Removed tick from unicode inputs.
@BramVer
Copy link
Contributor Author

BramVer commented Oct 7, 2018

@drashna Could you take a look at my current config? I've done most except for the unicode stuff, since I don't really get what needs to be changed.

@drashna
Copy link
Member

drashna commented Oct 7, 2018

Awesome, looks good!

@drashna drashna merged commit aa707c6 into qmk:master Oct 7, 2018
zer09 pushed a commit to zer09/qmk_firmware that referenced this pull request Oct 20, 2018
* Personal Nyquist keymap added.

* Responded to comments for PR.

Removed unnecessary imports.
Updated way of macros.
Removed tick from unicode inputs.

* formatting in keymap
tenderlove added a commit to tenderlove/qmk_firmware that referenced this pull request Jan 24, 2019
* master: (447 commits)
  Add documentation for Combo feature
  Cleanup learning newb links
  Update and cleanup AVR Hardware docs
  Keymap: DZ60 layout based on stephengrier's with PgUp/PgDn/Home/End, Insert, etc. (qmk#4110)
  evil80: Add readme.md file
  Added cheese keymap to tada68. (qmk#4105)
  Orange75: cleanup and Configurator support update (qmk#4101)
  Keymaps: Helix rgblight mode update (qmk#4091)
  Massdrop keyboards console device support for hid_listen
  Specify max layers supported in layer keycodes
  Update default Planck layout to be more in line with standard conventions
  Fix formatting in Unicode text
  Added link to One Shot Keys
  Added OSL to list of layer toggles
  Keymap: Iris Keymap  (qmk#4100)
  Keyboard: add orange75 (qmk#4093)
  DC01 updates and I2C avr speed overwrite (qmk#4088)
  Update Bluetooth docs (qmk#4062)
  feat: Add 68Keys.io
  Keymap: Personal Nyquist keymap added. (qmk#4059)
  ...
rseymour pushed a commit to rseymour/qmk_firmware that referenced this pull request Mar 13, 2019
* Personal Nyquist keymap added.

* Responded to comments for PR.

Removed unnecessary imports.
Updated way of macros.
Removed tick from unicode inputs.

* formatting in keymap
yamad pushed a commit to yamad/qmk_firmware that referenced this pull request Apr 10, 2019
* Personal Nyquist keymap added.

* Responded to comments for PR.

Removed unnecessary imports.
Updated way of macros.
Removed tick from unicode inputs.

* formatting in 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.

2 participants