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

several improvements for mitosis:datagrok #1960

Merged
merged 16 commits into from
Nov 3, 2017
Merged

several improvements for mitosis:datagrok #1960

merged 16 commits into from
Nov 3, 2017

Conversation

datagrok
Copy link
Contributor

@datagrok datagrok commented Nov 3, 2017

  • added readme with layout image and link to keyboard-layout-editor source
  • reformatted source code to comply with style guide (comments, indents)
  • added compatibility with (my) 8Mhz Pro Micro controller, and instructions to switch to 16Mhz
  • tweaked modifiers layout for easier access to function layer
  • added hack to move ? and !, keeping more punctuation and quotes on base layers
  • added toggle-able qwerty layer
  • made LED indicator update code more efficient (I think...)
  • moved verbose code comments to readme where appropriate

I'm concerned that my changes for the use of an 8Mhz Pro Micro controller might not be appropriate for the keymaps subdirectory. They are more of a hardware option that can be applied to the higher level keyboard files; they could be useful for boards using the other keymaps too. But I was unsure about the preferred way to set up a toggle for a hardware variant like that. Happy to implement recommendations either for this PR or as a follow-up.

This requires a corresponding change to the mitosis wireless firmware:
reversebias/mitosis#10
This is experimental, but compiles and seems to work correctly.
…d code

instead of [, .] [? !], using [, ?] [. !] greatly simplifies the code
needed to perform the shifted-key switching. (And keeps , and . on the
same keys that they are under qwerty.)

also: layout improvements for symbols
need to see if the corresponding changes needed in the keyboard-level code
is okay.
@jackhumbert jackhumbert merged commit e899cb8 into qmk:master Nov 3, 2017
@jackhumbert
Copy link
Member

Cool! Thanks :)

kgwong pushed a commit to kgwong/qmk_firmware that referenced this pull request Nov 26, 2017
* mitosis/datagrok: reduce features from rules.mk

* mitosis/datagrok: make both layer keys neighbor shift

* mitosis/datagrok: (no-op) tweak some comments

* mitosis/datagrok: set baudrate to 250k

This requires a corresponding change to the mitosis wireless firmware:
reversebias/mitosis#10

* mitosis/datagrok: move design description from code comment to a readme

* mitosis/datagrok: new layout, new shifted keys, efficient LED code

This is experimental, but compiles and seems to work correctly.

* mitosis/datagrok: whoops, move readme.md

* mitosis/datagrok: a minor layout improvement simplifies custom-shifted code

instead of [, .] [? !], using [, ?] [. !] greatly simplifies the code
needed to perform the shifted-key switching. (And keeps , and . on the
same keys that they are under qwerty.)

also: layout improvements for symbols

* mitosis/datagrok: make my code conform to QMK style guidelines

* mitosis/datagrok: TODO note for layout table in README

* mitosis/datagrok: remove led_set_user until i figure out other changes

need to see if the corresponding changes needed in the keyboard-level code
is okay.

* mitosis/datagrok: simpler layer indicator

* mitosis/datagrok: undo change to keyboard baud; make it in my layout dir.

* mitosis/datagrok: apply same punctuation hack to qwerty layer

* mitosis/datagrok: enable qwerty layer toggle

* mitosis/datagrok: update readme
akatrevorjay added a commit to akatrevorjay/qmk_firmware that referenced this pull request Apr 3, 2018
* 'master' of git://github.com/qmk/qmk_firmware: (291 commits)
  More "oscillope" keymap fixes. (qmk#1982)
  Improved README of yuuki and added RGB commands (qmk#1983)
  restructure converters (qmk#1825)
  qwerty_code_friendly: configurable left thumb
  Migrated most code from keymaps to userspace  (qmk#1980)
  Small ergodox config fix and update.
  Update and move around drashna keymaps (qmk#1976)
  make it easy to customize logo image
  add pgm_read_dword for Infinity ErgoDox
  333fred layout update (qmk#1971)
  Pete's 40th XD64 Layout
  Ergodox EZ and Atreus 42 key dvorak layout updates (qmk#1964)
  Correct the rules.mk documentation for auto shift.
  Fix RGBLIGHT startup color (qmk#1975)
  adds indication up to layer 7
  restore default mode/color if no 0 color
  adds per-layer rgb color option to ez
  update gh60 info
  add gh60 info.json
  several improvements for mitosis:datagrok (qmk#1960)
  ...
LovesTha pushed a commit to LovesTha/qmk_firmware that referenced this pull request Jul 24, 2018
* mitosis/datagrok: reduce features from rules.mk

* mitosis/datagrok: make both layer keys neighbor shift

* mitosis/datagrok: (no-op) tweak some comments

* mitosis/datagrok: set baudrate to 250k

This requires a corresponding change to the mitosis wireless firmware:
reversebias/mitosis#10

* mitosis/datagrok: move design description from code comment to a readme

* mitosis/datagrok: new layout, new shifted keys, efficient LED code

This is experimental, but compiles and seems to work correctly.

* mitosis/datagrok: whoops, move readme.md

* mitosis/datagrok: a minor layout improvement simplifies custom-shifted code

instead of [, .] [? !], using [, ?] [. !] greatly simplifies the code
needed to perform the shifted-key switching. (And keeps , and . on the
same keys that they are under qwerty.)

also: layout improvements for symbols

* mitosis/datagrok: make my code conform to QMK style guidelines

* mitosis/datagrok: TODO note for layout table in README

* mitosis/datagrok: remove led_set_user until i figure out other changes

need to see if the corresponding changes needed in the keyboard-level code
is okay.

* mitosis/datagrok: simpler layer indicator

* mitosis/datagrok: undo change to keyboard baud; make it in my layout dir.

* mitosis/datagrok: apply same punctuation hack to qwerty layer

* mitosis/datagrok: enable qwerty layer toggle

* mitosis/datagrok: update readme
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants