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

Replace RotaryNavConfig with RotaryInputAdapter #206

Merged
merged 11 commits into from
Sep 17, 2024

Conversation

forntoh
Copy link
Owner

@forntoh forntoh commented Sep 17, 2024

Replaced the RotaryNavConfig struct with a new RotaryInputAdapter class to handle rotary input for LCD menus efficiently. The adapter manages user input through a rotary encoder, recognizing short, long, and double presses for menu navigation and selection.

@forntoh forntoh added this to the LcdMenu 4.3.0 milestone Sep 17, 2024
@forntoh forntoh self-assigned this Sep 17, 2024
@forntoh forntoh enabled auto-merge (squash) September 17, 2024 22:21
src/input/RotaryInputAdapter.h Outdated Show resolved Hide resolved
src/input/RotaryInputAdapter.h Outdated Show resolved Hide resolved
src/input/RotaryInputAdapter.h Outdated Show resolved Hide resolved
@ShishkinDmitriy
Copy link
Collaborator

I did the same changes)

Replaced the RotaryNavConfig struct with a new RotaryInputAdapter class to handle rotary input for LCD menus efficiently. The adapter manages user input through a rotary encoder, recognizing short, long, and double presses for menu navigation and selection.
- Removed explicit longPressDuration and doublePressThreshold settings.
- Added default values for LONG_PRESS_DURATION and DOUBLE_PRESS_THRESHOLD in constants.h.
- Renamed `RotaryInputAdapter` to `SimpleRotaryAdapter`
- Updated references in code and file names to reflect the change.
@forntoh forntoh force-pushed the tf/replace-rotary-nav-config branch from 6e6d9fb to 71b47d9 Compare September 17, 2024 23:13
@forntoh forntoh disabled auto-merge September 17, 2024 23:52
@forntoh forntoh merged commit afeaf80 into feature/move-control-to-items Sep 17, 2024
6 checks passed
@forntoh forntoh deleted the tf/replace-rotary-nav-config branch September 17, 2024 23:53
@forntoh forntoh added the input adapter Add support for a new input method (device) label Sep 20, 2024
@forntoh forntoh added the feature New feature label Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature input adapter Add support for a new input method (device)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants