Skip to content

[Feature] Keyboard controls#5

Open
castorNova2 wants to merge 3 commits intosugarlabs:mainfrom
castorNova2:main
Open

[Feature] Keyboard controls#5
castorNova2 wants to merge 3 commits intosugarlabs:mainfrom
castorNova2:main

Conversation

@castorNova2
Copy link
Copy Markdown

@castorNova2 castorNova2 commented Apr 12, 2026

Keyboard control added to move tile using arrow keys(Up, Down, Left, Right)

Video

Screencast_20260415_184658.webm

@castorNova2
Copy link
Copy Markdown
Author

@chimosky Please review the changes.

@Bishoywadea Bishoywadea self-requested a review April 15, 2026 04:06
@Bishoywadea Bishoywadea self-assigned this Apr 15, 2026
@Bishoywadea
Copy link
Copy Markdown
Collaborator

@castorNova2 hello,
In addition, I personally don’t see much value in adding keyboard controls since there is only one possible move at a time (you can’t move a tile in more than one direction per play), but that’s fine.
The issue is that I’m not able to move it using the keyboard. Did you test this change before opening the PR? If it’s working on your side, could you please record a video so I can check whether the problem is on my side?

@castorNova2
Copy link
Copy Markdown
Author

castorNova2 commented Apr 15, 2026

@castorNova2 hello, In addition, I personally don’t see much value in adding keyboard controls since there is only one possible move at a time (you can’t move a tile in more than one direction per play), but that’s fine. The issue is that I’m not able to move it using the keyboard. Did you test this change before opening the PR? If it’s working on your side, could you please record a video so I can check whether the problem is on my side?

Yes I have tested on my sugar OS VM, controls are working as intended. I will attach a video to the PR desciption.

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