✅getDirection() returns1 for clockwise,-1 for counter-clockwise, and0 when no movement is detected (debounced).
🧠 Internal debouncing is handled viaDEBOUNCE_DELAY to avoid noisy reads.
🛠️ Thebegin() logic now initializes internal state (_lastPosition,_lastDebounceTime) accordingly.
🧪 Thebasic example sketch has been updated to demonstrate the use ofgetDirection().

🔁 Related Work

✔️ Brings over logic and fixes from PRModulinoKnob: report error and not last valid _pressed if read() fails #20, adapting them for this implementation.
🔗 Closes issueAdd function for Modulino Knob to detect direction #9 by properly implementing movement direction detection and showing how it can be used in real code, as discussed inthis comment.

Fixed#9

AntonioBerna added5 commits

April 30, 2025 03:41

feat(buttons): allow isPressed() to accept 'A', 'B', 'C' as input

53c079c

Added support for character and string-based input to `isPressed()` in `ModulinoButtons`,so it now accepts both index (0–2) and letter identifiers (`'A'`, `'B'`, `'C'`) matchingthe physical button labeling.Closesarduino-libraries#3

Update example to document support for both numeric and letter indice…

59aef64

…s ♻️

Add getDirection() support to ModulinoKnob and example ⚡

e53f803

Fix: 🔥

9a2a9cf

Fix: restore files 🔥

3d48585

per1234 added the enhancementNew feature or request label

May 2, 2025

leonardocavagnis self-requested a review

May 8, 2025 14:08

leonardocavagnis requested changes

May 8, 2025

View reviewed changes

Copy link

Contributor

leonardocavagnis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Thanks for contribution!
Please, take a look to my review. Minor fixes

.gitignore Outdated

Copy link

Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Remove this file.
It is already included in the PR#33

examples/Modulino_Buttons/Buttons_Basic/Buttons_Basic.ino

Copy link

Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Revert this file :)
this modifies doesn't add any feature to the PR

examples/Modulino_Knob/Knob_Basic/Knob_Basic.ino OutdatedShow resolvedHide resolved

src/Modulino.h

		}
		return ret;
		}
		int16_tget() {

Copy link

Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

remove it.
it is already included in the PR#32

src/Modulino.h Outdated

		if (res ==false) {
		return0;
		}
		get(buf);

Copy link

Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

same here
it is already included in the PR#32

AntonioBerna added3 commits

May 8, 2025 17:04

Minor fixes ♻️

c70097d

Update API docs ♻️

66200f1

Update API docs ♻️

cff95ab

Copy link

ContributorAuthor

AntonioBerna commentedMay 8, 2025

@leonardocavagnis I apologize for creating problems with the files but I had accidentally merged the changes of PR#33 into themain branch of theModulino repository fork and when I created the branch for PR#34 the changes were automatically added.

With commit66200f1 this PR is affected topull request is merged indirectly or is still processing updates.

For this reason with commitcff95ab everything should be fixed correctly.

I remain waiting for further developments. Enjoy 🎉