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

(enh) Music editor: Skip over sharp column when using cursor keys #1672

Merged
merged 1 commit into from
Nov 6, 2021

Conversation

joshgoebel
Copy link
Collaborator

Allows toggling of sharp. I guess technically it doesn't even require you to be in the column...

Resolves #1625.

@runlow
Copy link

runlow commented Nov 5, 2021

I still think it should not be possible to even "be in the column" if being in it doesn't do anything.

@joshgoebel joshgoebel changed the title WIP: toggle sharp with keyboard . WIP: toggle sharp with ., skip sharp column with cursor Nov 5, 2021
@joshgoebel joshgoebel marked this pull request as draft November 5, 2021 22:52
@joshgoebel joshgoebel changed the title WIP: toggle sharp with ., skip sharp column with cursor (enh) Music editor: Skip over sharp column when using cursor keys Nov 6, 2021
@joshgoebel
Copy link
Collaborator Author

Rebased to only include the skipping behavior, which feels very nice and natural. (and tiny PR)

Copy link
Owner

@nesbox nesbox left a comment

Choose a reason for hiding this comment

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

Looks good, thank you

@nesbox
Copy link
Owner

nesbox commented Nov 6, 2021

Please disable Draft mode for the PR if it's ready

@joshgoebel joshgoebel marked this pull request as ready for review November 6, 2021 20:15
@joshgoebel
Copy link
Collaborator Author

Sorry!

@nesbox nesbox merged commit c5c2b7b into nesbox:master Nov 6, 2021
@runlow
Copy link

runlow commented Nov 6, 2021

Thank you!

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.

tracker "sharp column" should be editable or not selectable
3 participants