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.

Sign up for GitHub

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

Resolve conflict merging master to develop #18297

Merged
merged 3 commits into from
Sep 6, 2022
Merged

Resolve conflict merging master to develop #18297

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
60cac6d
Discourage use of ENCODER_MAP at keyboard level (#18286)
zvecr Sep 6, 2022
7160487
Fungo rev1: fix QMK Configurator key sequence (#18293)
noroadsleft Sep 6, 2022
b1a838f
Merge branch 'develop' into merge_conflict_resolution
spidey3 Sep 6, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion docs/feature_encoders.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ Additionally, if one side does not have an encoder, you can specify `{}` for the

## Encoder map :id=encoder-map

Encoder mapping may be added to your `keymap.c`, which replicates the normal keyswitch layer handling functionality, but with encoders. Add this to your `rules.mk`:
Encoder mapping may be added to your `keymap.c`, which replicates the normal keyswitch layer handling functionality, but with encoders. Add this to your keymap's `rules.mk`:

```make
ENCODER_MAP_ENABLE = yes
Expand All @@ -88,6 +88,8 @@ const uint16_t PROGMEM encoder_map[][NUM_ENCODERS][2] = {
#endif
```

?> This should only be enabled at the keymap level.

Using encoder mapping pumps events through the normal QMK keycode processing pipeline, resulting in a _keydown/keyup_ combination pushed through `process_record_xxxxx()`. To configure the amount of time between the encoder "keyup" and "keydown", you can add the following to your `config.h`:

```c
Expand Down
3 changes: 3 additions & 0 deletions docs/pr_checklist.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,9 @@ https://github.com/qmk/qmk_firmware/pulls?q=is%3Apr+is%3Aclosed+label%3Akeyboard
- no `(-/+size)` comments related to enabling features
- remove the list of alternate bootloaders if one has been specified
- no re-definitions of the default MCU parameters if same value, when compared to the equivalent MCU in [mcu_selection.mk](https://github.com/qmk/qmk_firmware/blob/master/builddefs/mcu_selection.mk)
- no "keymap only" features enabled
- `COMBO_ENABLE`
- `ENCODER_MAP_ENABLE`
- keyboard `config.h`
- don't repeat `MANUFACTURER` in the `PRODUCT` value
- no `#define DESCRIPTION`
Expand Down
168 changes: 77 additions & 91 deletions keyboards/fungo/rev1/info.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,101 +11,87 @@
"layouts": {
"LAYOUT_fungo_split_num": {
"layout": [
{"x": 0, "y": 0.375},
{"x": 1, "y": 0.375},
{"x": 2, "y": 0.125},
{"x": 3, "y": 0},
{"x": 4, "y": 0.125},
{"x": 5, "y": 0.25},
{"x": 6, "y": 0.5},
{"x":0, "y":0.375},
{"x":1, "y":0.375},
{"x":2, "y":0.125},
{"x":3, "y":0},
{"x":4, "y":0.125},
{"x":5, "y":0.25},
{"x":6, "y":0.5},
{"x":12, "y":0.5},
{"x":13, "y":0.25},
{"x":14, "y":0.125},
{"x":15, "y":0},
{"x":16, "y":0.125},
{"x":17, "y":0.375},
{"x":18, "y":0.375},

{"x": 12, "y": 0.5},
{"x": 13, "y": 0.25},
{"x": 14, "y": 0.125},
{"x": 15, "y": 0},
{"x": 16, "y": 0.125},
{"x": 17, "y": 0.375},
{"x": 18, "y": 0.375},

{"x":0, "y":1.375},
{"x":1, "y":1.375},
{"x":2, "y":1.125},
{"x":3, "y":1},
{"x":4, "y":1.125},
{"x":5, "y":1.25},
{"x":6, "y":1.5},
{"x":12, "y":1.5},
{"x":13, "y":1.25},
{"x":14, "y":1.125},
{"x":15, "y":1},
{"x":16, "y":1.125},
{"x":17, "y":1.375},
{"x":18, "y":1.375},

{"x": 0, "y": 1.375},
{"x": 1, "y": 1.375},
{"x": 2, "y": 1.125},
{"x": 3, "y": 1},
{"x": 4, "y": 1.125},
{"x": 5, "y": 1.25},
{"x": 6, "y": 1.5},
{"x":0, "y":2.375},
{"x":1, "y":2.375},
{"x":2, "y":2.125},
{"x":3, "y":2},
{"x":4, "y":2.125},
{"x":5, "y":2.25},
{"x":6, "y":2.5},
{"x":12, "y":2.5},
{"x":13, "y":2.25},
{"x":14, "y":2.125},
{"x":15, "y":2},
{"x":16, "y":2.125},
{"x":17, "y":2.375},
{"x":18, "y":2.375},

{"x": 12, "y": 1.5},
{"x": 13, "y": 1.25},
{"x": 14, "y": 1.125},
{"x": 15, "y": 1},
{"x": 16, "y": 1.125},
{"x": 17, "y": 1.375},
{"x": 18, "y": 1.375},

{"x": 0, "y": 2.375},
{"x": 1, "y": 2.375},
{"x": 2, "y": 2.125},
{"x": 3, "y": 2},
{"x": 4, "y": 2.125},
{"x": 5, "y": 2.25},
{"x": 6, "y": 2.5},
{"x":0, "y":3.375},
{"x":1, "y":3.375},
{"x":2, "y":3.125},
{"x":3, "y":3},
{"x":4, "y":3.125},
{"x":5, "y":3.25},
{"x":6, "y":3.5},
{"x":7, "y":4},
{"x":8.25, "y":3.75},
{"x":9.75, "y":3.75},
{"x":11, "y":4},
{"x":12, "y":3.5},
{"x":13, "y":3.25},
{"x":14, "y":3.125},
{"x":15, "y":3},
{"x":16, "y":3.125},
{"x":17, "y":3.375},
{"x":18, "y":3.375},

{"x": 12, "y": 2.5},
{"x": 13, "y": 2.25},
{"x": 14, "y": 2.125},
{"x": 15, "y": 2},
{"x": 16, "y": 2.125},
{"x": 17, "y": 2.375},
{"x": 18, "y": 2.375},

{"x": 0, "y": 3.375},
{"x": 1, "y": 3.375},
{"x": 2, "y": 3.125},
{"x": 3, "y": 3},
{"x": 4, "y": 3.125},
{"x": 5, "y": 3.25},
{"x": 6, "y": 3.5},

{"x": 12, "y": 3.5},
{"x": 13, "y": 3.25},
{"x": 14, "y": 3.125},
{"x": 15, "y": 3},
{"x": 16, "y": 3.125},
{"x": 17, "y": 3.375},
{"x": 18, "y": 3.375},

{"x": 0, "y": 4.375},
{"x": 1, "y": 4.375},
{"x": 2, "y": 4.25},
{"x": 3, "y": 4.25},
{"x": 5.25, "y": 3},
{"x": 5.75, "y": 5},
{"x": 7, "y": 4},

{"x": 11, "y": 4},
{"x": 13.25, "y": 5},
{"x": 13.75, "y": 3},
{"x": 15, "y": 4.25},
{"x": 16, "y": 4.25},
{"x": 17, "y": 4.375},
{"x": 18, "y": 4.375},


{"x": 7, "y": 5},
{"x": 8.25, "y": 3.75},
{"x": 8.25, "y": 4.75},

{"x": 9.75, "y": 4.75},
{"x": 9.75, "y": 3.75},
{"x": 11, "y": 5},





]
{"x":0, "y":4.375},
{"x":1, "y":4.375},
{"x":2, "y":4.25},
{"x":3, "y":4.25},
{"x":4.333, "y":4.25},
{"x":5.667, "y":4.625},
{"x":7, "y":5},
{"x":8.25, "y":4.75},
{"x":9.75, "y":4.75},
{"x":11, "y":5},
{"x":12.333, "y":4.625},
{"x":13.667, "y":4.25},
{"x":15, "y":4.25},
{"x":16, "y":4.25},
{"x":17, "y":4.375},
{"x":18, "y":4.375}
]
}
}
}