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

Custom Shifts #2

Conversation

IsaacElenbaas
Copy link

@IsaacElenbaas IsaacElenbaas commented Jul 29, 2020

Depends on #1. The pressing and releasing of autoshift keys is done with two functions, passed a keycode and whether they should be shifted (release needs it for if you have a different base key for the shifted value). There is also a function to simply add keys to AS, which can also be used for other things (required for mod-taps that autoshift, for example) if needed.

@IsaacElenbaas

This comment has been minimized.

@IsaacElenbaas

This comment has been minimized.

@IsaacElenbaas IsaacElenbaas marked this pull request as draft July 30, 2020 17:15
@IsaacElenbaas
Copy link
Author

Working on some stuff with the functions to make you need less and them give more info.

@IsaacElenbaas
Copy link
Author

I'm happy with where it's at now, but leaving as draft at least until/if cleanup is merged. Still open to ideas to better the functions, and I'm not sure if the docs are clear/concise enough.

@IsaacElenbaas
Copy link
Author

IsaacElenbaas commented Jul 31, 2020

Well, I tried to resolve the conflicts here as well, but now GitHub just says they're "too complex to resolve in the web editor" lol
EDIT: Had done them correctly and GitHub just didn't know it until cleanup was merged 😎

@IsaacElenbaas
Copy link
Author

With tap-hold action support being added here, I don't see a use for autoshift_is_custom having everything passed to it. Docs can also be slimmed down a lot with the removal of the mod-tap example. I'll do that later today and make sure this doesn't have merge conflicts.

@IsaacElenbaas IsaacElenbaas marked this pull request as ready for review August 7, 2020 10:53
noroadsleft and others added 18 commits August 11, 2020 01:23
* Initial work for consolidation of board files and default ChibiOS configs.

* Migrate F401/F411 black pills for testing.

* Add early init bootloader jump flag.

* Add support for I2C in order to use i2c_scanner keymap.

* Add F401/F411 HSE bypass to get things booting.

* Exempt "hooked" ChibiOS conf files from updater script.

* Fix up ordering for bootloader_defs file check.

* Match previous $(KEYBOARD_PATHS) value for Proton-C, updated for all board configs.
* Change analogRead calls to analogReadPin

* Add ChangeLog

* Update docs, remove mention of `analogRead()`

* Retarget changelog for next round
Mousekey scrolling should have a separate repeat variable
to keep track of scrolling acceleration, instead of being
tied to mouse movement scolling in mousekeys. The send function
should record when the last movement was made since this is
when movement is actually sent. Doing this fixes the bug where
the initial press of a mousekey scroll button causes a double scroll.

Signed-off-by: Daniel Hong <daniel.hong@live.com>
* Add change log

* Change combo index from uint8_t to uint16_t
* Re-fix the dual-bank bootloader stuff.

* Use wait_ms() instead of using nop's for a delay, as ChibiOS is actually running at the time of bootloader jump.
* Disable NKRO on V-USB controllers

* not _currently_ supported text

Co-authored-by: Ryan <fauxpark@gmail.com>

Co-authored-by: Ryan <fauxpark@gmail.com>
@p00ya p00ya force-pushed the pull-auto-shift branch 2 times, most recently from 2e114f8 to 83a090d Compare August 31, 2020 00:19
p00ya pushed a commit that referenced this pull request Nov 14, 2020
* Add Initial layout configuration.

* Set arduino pro micro's built in RX LED as Numlock

* Enable Autoshift feautre.

* Remove DVORAK layout.

* Add Numpad, update default layer.

* Update AutoShift timeout.

* Beautify ASCII Art.

* Update keymap.

* Add Initial verison of RGB lighting.

* Add Tap dance, hungarian letters.

.Xmodmap example: geripgeri/dotfiles@09f0587

* Update RGB to use only noeeprom functions.

* Change default rgb led color.

* Add `pragma once`.

* Emojis added.

* Add tap dance for hungarian letters, multimedia control, slash/backslash.

* Remove audio related code.

* Add Gaming layer.

* Remove unnecessary code, update copyright.

* Add readme.md

* Implement code review suggestions.

* Implement code review suggestions #2

* Remove Auto Shift feature.

* Remove Gaming layer.
@IsaacElenbaas
Copy link
Author

Will be reopened into develop.

@IsaacElenbaas IsaacElenbaas deleted the as_custom_shifts_keyrepeat branch February 23, 2022 15:39
p00ya pushed a commit that referenced this pull request Jan 14, 2023
)

* Replace Tapping Force Hold feature with Quick Tap Term

* Replace keyboard level TAPPING_FORCE_HOLD with QUICK_TAP_TERM 0

* Deprecate force hold in info_config.json

* Before and after quick tap term unit tests

* Quick tap unit tests iteration

* Keymap config.h correction

* Remove TAPPING_FORCE_HOLD_PER_KEY macros that were missed

* Add two more test cases for quick tap

* Replace TAPPING_FORCE_HOLD with QUICK_TAP_TERM in configs #2

* Replace TAPPING_FORCE_HOLD_PER_KEY with QUICK_TAP_TERM_PER_KEY in configs #2

* Add function declaration for get_quick_tap_term

Co-authored-by: Stefan Kerkmann <karlk90@pm.me>
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.