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

Allow edge keys instead of corner keys (swipe vertically/horizontally) #57

Merged
merged 1 commit into from
Feb 6, 2022

Conversation

MaxGyver83
Copy link
Contributor

Fix/implement #56.

Add a new boolean parameter "edgekeys" for defining keys that have the additional (swipe) keys on the edges (top, right, left, bottom) instead of at the corners (top left, top right, bottom left, bottom right).

Example:

<key width="1.2" key1="up" key2="right" key3="left" key4="down" edgekeys="true"/>

If the edgekeys parameter is omitted, it defaults to false (swipe diagonally for additional keys).

This pull request also changes the arrow keys in all layouts to the alternative edge style:

Before:
Arrow key

After:
Arrow key edges

Add a new boolean parameter "edgekeys" for defining keys that have the
additional (swipe) keys on the edges (top, right, left, bottom) instead
of at the corners (top left, top right, bottom left, bottom right).
@MaxGyver83 MaxGyver83 changed the title Allow egde keys instead of corner keys (swipe vertically/horizontally) Allow edge keys instead of corner keys (swipe vertically/horizontally) Feb 2, 2022
@Julow
Copy link
Owner

Julow commented Feb 6, 2022

I've used this since yesterday and it's working great :) I do prefer the arrows now, though I don't use them much.
Thanks a lot !

@avently mentioned that the arrow keys were already fine but I think vertical/horizontal movements are easier than diagonals at the bottom of the screen in addition to being more intuitive. I'm also investigating improving cursor movements and this will help a lot.

@Julow Julow merged commit 93edc4a into Julow:master Feb 6, 2022
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.

2 participants