Skip to content

Sticky keys - #118

Merged
PaulFreund merged 1 commit into
SynthstromAudible:communityfrom
sapphire-arches:sticky-keys
Sep 2, 2023
Merged

Sticky keys#118
PaulFreund merged 1 commit into
SynthstromAudible:communityfrom
sapphire-arches:sticky-keys

Conversation

@sapphire-arches

@sapphire-arches sapphire-arches commented Jun 28, 2023

Copy link
Copy Markdown
Collaborator

Fixes #107

TODO:

  • Long press enable mode
  • Illuminate shift when it's stuck on by sticky keys
  • Audit the various test modes to make sure they will work with stickykeys (either by disabling it or adding appropriate calls to the shift mode code)
  • Custom UI class for the sticky keys menu (7seg compat + long press enable/time configuration)
  • Add audible indication when sticky keys enabled (if it's easy, this is a very silly feature)

@sapphire-arches sapphire-arches mentioned this pull request Jun 28, 2023
@sapphire-arches
sapphire-arches force-pushed the sticky-keys branch 3 times, most recently from 6e51b91 to 3423567 Compare June 29, 2023 02:22
@sapphire-arches
sapphire-arches marked this pull request as ready for review June 29, 2023 02:23
@sapphire-arches

Copy link
Copy Markdown
Collaborator Author

I'm probably not going to implement the sound notification, as funny as it would be. I don't have the MIDI hardware to test CC learning either, so I'll leave that for a followup.

@sapphire-arches

Copy link
Copy Markdown
Collaborator Author

Known bug: if you disable sticky keys with the shift key in sticky mode, it'll stay in sticky mode until a reset.

@litui litui added the enhancement New feature or request label Jun 30, 2023
@jamiefaye

Copy link
Copy Markdown
Collaborator

Please fix the conflicts and we can merge it in.

@sapphire-arches

sapphire-arches commented Jul 3, 2023 via email

Copy link
Copy Markdown
Collaborator Author

@jamiefaye

Copy link
Copy Markdown
Collaborator

Waiting here for conflicts to be resolved.

@jamiefaye jamiefaye added the merge conflict github reports a merge conflict that should be resolved. label Jul 12, 2023
@sapphire-arches
sapphire-arches marked this pull request as draft July 25, 2023 22:26
@wtkdwc

wtkdwc commented Aug 22, 2023

Copy link
Copy Markdown

hi, just an thought (no clue about implementation) - wouldn't be more practical the opposite behavior of turning sticky on? like on novation circuit menus - one very short press/click turns on sticky shift and holding longer was standard behavior. that way you would avoid mentioned accidental turning it on or having to wait longer times to turn it on.

@PaulFreund PaulFreund added the stale PR was not updated in two weeks and will be closed soon. A new PR can be created to pick up the work label Aug 25, 2023
@PaulFreund

Copy link
Copy Markdown
Collaborator

Hey, a quick heads up :) After two weeks without any activity from the author of a PR we tag it with the "stale" label and close it after another week of inactivity as per our Contributor guidelins. This is not to create pressure or discourage anyone but to keep the list of Pull requests maintainable.

We are very happy about your contribution and nothing will be lost. You can pick up the work at any time by reopening the PR or creating a new one.

@sapphire-arches
sapphire-arches marked this pull request as ready for review August 29, 2023 05:25
This implements a "sticky keys" function for the shift key, which
toggles on short press. It also wires up the shift LED to give feedback
when sticky keys is enabled. This can be overriden in case the user
doesn't want it (and can be enabled separately if they want feedback
even when sticky keys isn't enabled).

Fixes SynthstromAudible#107
@sapphire-arches

Copy link
Copy Markdown
Collaborator Author

Thanks for the idea @wtkdwc, I've implemented it exactly that way. When sticky shift is on, a short press will stick shift on, unless you also press another button during the short press (we assume that means you wanted to just quickly enter a combo). Short press again will disable it.

@PaulFreund PaulFreund added Code Review wanted and removed stale PR was not updated in two weeks and will be closed soon. A new PR can be created to pick up the work labels Sep 2, 2023
@PaulFreund
PaulFreund added this pull request to the merge queue Sep 2, 2023
Merged via the queue into SynthstromAudible:community with commit 9e4254a Sep 2, 2023
PaulFreund pushed a commit that referenced this pull request Sep 13, 2023
This implements a "sticky keys" function for the shift key, which
toggles on short press. It also wires up the shift LED to give feedback
when sticky keys is enabled. This can be overriden in case the user
doesn't want it (and can be enabled separately if they want feedback
even when sticky keys isn't enabled).

Fixes #107
@sdesign75

Copy link
Copy Markdown

In clip view you can’t audition notes when sticky shift is enabled. Is there a way to still audition notes so I can hear my patch while using shortcuts to edit the sound parameters?

@duncanrmi

Copy link
Copy Markdown

crashes my machine before I can even exit the settings menu. :-(

@sapphire-arches

Copy link
Copy Markdown
Collaborator Author

@sdesign75 today there is not, we probably should have a way to do it.

@sapphire-arches

Copy link
Copy Markdown
Collaborator Author

@duncanrmi please open a new issue with a description of exactly what you were doing when the device crashed.

@duncanrmi

duncanrmi commented Nov 6, 2023 via email

Copy link
Copy Markdown

@sapphire-arches
sapphire-arches deleted the sticky-keys branch January 29, 2024 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Code Review wanted enhancement New feature or request merge conflict github reports a merge conflict that should be resolved.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Shift Lock

7 participants