Skip to content

PETKA: Add keymapper support#6860

Merged
sev- merged 1 commit intoscummvm:masterfrom
aunnoman1:petka-keymapper
Sep 1, 2025
Merged

PETKA: Add keymapper support#6860
sev- merged 1 commit intoscummvm:masterfrom
aunnoman1:petka-keymapper

Conversation

@aunnoman1
Copy link
Contributor

No description provided.

@OMGPizzaGuy OMGPizzaGuy added the GSoC Part of a Google Summer of Code project label Aug 6, 2025
@aunnoman1 aunnoman1 marked this pull request as ready for review August 8, 2025 23:16
Copy link
Member

@sev- sev- left a comment

Choose a reason for hiding this comment

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

Added my notes

act->addDefaultInputMapping("JOY_B");
engineKeymap->addAction(act);

act = new Action("LOOK", _("Switch cursor to 'look'"));
Copy link
Member

Choose a reason for hiding this comment

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

Unless you used this same wording in other engines, I’d recommend renaming this action to simple “Look”. Same for other places

act->addDefaultInputMapping("JOY_RIGHT_TRIGGER");
engineKeymap->addAction(act);

act = new Action("INVENTORY", _("Toggle inventory"));
Copy link
Member

Choose a reason for hiding this comment

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

What does “toggle” mean here? Either use better wording or add a note to translators

Copy link
Contributor Author

Choose a reason for hiding this comment

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

"toggle" here means you can open and close the inventory by pressing the same key. would "Show / hide inventory" be a better description?

Copy link
Member

Choose a reason for hiding this comment

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

Check the similar actions in other engines, how are they described there?

act->addDefaultInputMapping("JOY_RIGHT_SHOULDER");
engineKeymap->addAction(act);

act = new Action("OPTIONS", _("Toggle options screen"));
Copy link
Member

Choose a reason for hiding this comment

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

Isn’t it just “Options”?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

"toggle" here means you can open and close the options screeen by pressing the same key. would "Show / hide options screen" be a better description or simply "Options"?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, this sounds better

act->addDefaultInputMapping("F3");
engineKeymap->addAction(act);

act = new Action("FIXCURSOR", _("Fix cursor"));
Copy link
Member

Choose a reason for hiding this comment

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

What does fixing mean in this context? Explanation to the translators is required

Copy link
Contributor Author

Choose a reason for hiding this comment

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

During my testing, I did not have any issues with the cursor. Hence, i'm not sure what issue occurs.

However, There is a comment in the code regarding this and the function called seems to make the cursor draw again:

_mainInterface->_dialog.fixCursor(); // Buggy in original

@sev-
Copy link
Member

sev- commented Sep 1, 2025

Thank you!

@sev- sev- merged commit a84f448 into scummvm:master Sep 1, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GSoC Part of a Google Summer of Code project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants