From 79cc6ce2d0d860680834e9b0cadfbe42e789fa0c Mon Sep 17 00:00:00 2001 From: Drashna Jaelre Date: Fri, 25 Jun 2021 20:57:48 -0700 Subject: [Keyboard] Bugfixes and Readme additions for PloopyCo devices (#13043) --- keyboards/ploopyco/mouse/mouse.c | 25 ++++++------------------- keyboards/ploopyco/mouse/readme.md | 14 ++++++++++++++ 2 files changed, 20 insertions(+), 19 deletions(-) (limited to 'keyboards/ploopyco/mouse') diff --git a/keyboards/ploopyco/mouse/mouse.c b/keyboards/ploopyco/mouse/mouse.c index 1eb5e3ead9..0bf96a20f7 100644 --- a/keyboards/ploopyco/mouse/mouse.c +++ b/keyboards/ploopyco/mouse/mouse.c @@ -247,7 +247,6 @@ void pointing_device_init(void) { opt_encoder_init(); } -bool has_report_changed(report_mouse_t new, report_mouse_t old) { return (new.buttons != old.buttons) || (new.x && new.x != old.x) || (new.y && new.y != old.y) || (new.h && new.h != old.h) || (new.v && new.v != old.v); } void pointing_device_task(void) { report_mouse_t mouse_report = pointing_device_get_report(); @@ -256,7 +255,12 @@ void pointing_device_task(void) { if (is_drag_scroll) { mouse_report.h = mouse_report.x; +#ifdef PLOOPY_DRAGSCROLL_INVERT + // Invert vertical scroll direction + mouse_report.v = -mouse_report.y; +#else mouse_report.v = mouse_report.y; +#endif mouse_report.x = 0; mouse_report.y = 0; } @@ -265,27 +269,10 @@ void pointing_device_task(void) { pointing_device_send(); } -void pointing_device_send(void) { - static report_mouse_t old_report = {}; - report_mouse_t mouseReport = pointing_device_get_report(); - - // If you need to do other things, like debugging, this is the place to do it. - if (has_report_changed(mouseReport, old_report)) { - host_mouse_send(&mouseReport); - } - - // send it and 0 it out except for buttons, so those stay until they are explicity over-ridden using update_pointing_device - mouseReport.x = 0; - mouseReport.y = 0; - mouseReport.v = 0; - mouseReport.h = 0; - pointing_device_set_report(mouseReport); - old_report = mouseReport; -} - void eeconfig_init_kb(void) { keyboard_config.dpi_config = PLOOPY_DPI_DEFAULT; eeconfig_update_kb(keyboard_config.raw); + eeconfig_init_user(); } void matrix_init_kb(void) { diff --git a/keyboards/ploopyco/mouse/readme.md b/keyboards/ploopyco/mouse/readme.md index ff37a6b807..e9ad915a66 100644 --- a/keyboards/ploopyco/mouse/readme.md +++ b/keyboards/ploopyco/mouse/readme.md @@ -50,3 +50,17 @@ To configure/set your own array, there are two defines to use, `PLOOPY_DPI_OPTIO The `PLOOPY_DPI_OPTIONS` array sets the values that you want to be able to cycle through, and the order they are in. The "default" define lets the firmware know which of these options is the default and should be loaded by default. The `DPI_CONFIG` macro will cycle through the values in the array, each time you hit it. And it stores this value in persistent memory, so it will load it the next time the device powers up. + +## Drag Scroll + +Drag Sroll is a custom keycode for the Ploopy devices that allow you to hold or tap a button and have the mouse movement translate into scrolling instead. + +Nothing needs to be enabled to use this functionality. Just add the `DRAG_SCROLL` to your keymap. + +### Drag Scroll Configuration + +* `#define PLOOPY_DRAGSCROLL_MOMENTARY` - Makes the key into a momentary key, rather than a toggle. +* `#define PLOOPY_DRAGSCROLL_MULTIPLIER 0.75` - Sets the DPI multiplier to use when drag scroll is enabled. +* `#define PLOOPY_DRAGSCROLL_FIXED` - Normally, when activating Drag Scroll, it uses a fraction of the current DPI. You can define this to use a specific, set DPI rather than a fraction of the current DPI. + * `#define PLOOPY_DRAGSCROLL_DPI 100` - When the fixed DPI option is enabled, this sets the DPI to be used for Drag Scroll. +* `#define PLOOPY_DRAGSCROLL_INVERT` - This reverses the direction that the scroll is performed. -- cgit v1.2.3