Skip to content

Two players on a single keyboard - #57

Merged
mupfdev merged 5 commits into
ngagesdk:mainfrom
cherepets:two_players_same_kbd
Aug 24, 2026
Merged

mupfdev merged 5 commits into
ngagesdk:mainfrom
cherepets:two_players_same_kbd

Conversation

@cherepets

@cherepets cherepets commented Aug 23, 2026

Copy link
Copy Markdown
Collaborator
  • I confirm that I am the author of this code and release it to the open8 project under the MIT license. This contribution does not contain code from other sources, including code generated by a Large Language Model ("AI").

Description

On desktop, use keyboard mappings from https://pico-8.fandom.com/wiki/Btn to allow multiplayer with a single keyboard.
Tested on https://www.lexaloffle.com/bbs/?tid=45788 on Windows, not sure how should it work on Symbian (likely not needed there).
Also added support theoretical support for 2 touch players (I need it for a different platform).

Existing Issue(s)

It wasn't possible to play pong on a single keyboard before

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 10 complexity · 0 duplication

Metric Results
Complexity 10
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@mupfdev

mupfdev commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Looks good, thank you!

PS: Could you please add brackets to the if-clauses, as this makesthe code clearer and helps prevent potential errors when the code is modified later?

@mupfdev mupfdev added the enhancement New feature or request label Aug 24, 2026
@cherepets

Copy link
Copy Markdown
Collaborator Author

Sure, added

@mupfdev
mupfdev merged commit b95bf96 into ngagesdk:main Aug 24, 2026
15 of 17 checks passed
@cherepets
cherepets deleted the two_players_same_kbd branch August 24, 2026 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants