Skip to content

Conversation

@Ioxcv12
Copy link

@Ioxcv12 Ioxcv12 commented Jun 25, 2025

Parallax [unfortunately for Kittopia] uses Crtl + P, and I don't think Linx is willing to change the shortcut, so I have made this pull request to request to change it.

@R-T-B
Copy link
Member

R-T-B commented Jun 25, 2025

I'm willing to do this but am strapped for time this week, so you might not see a build for a few days. But I have no issues with the idea, anyways.

@StollD
Copy link
Member

StollD commented Jun 27, 2025

Maybe pressing Ctrl + P should open a popup telling the user that the shortcut has been changed, and then write a file into GameData that prevents the popup from opening again?

(That said, Kittopia has used Ctrl + P for longer than Kopernicus even exists ...)

@Ioxcv12
Copy link
Author

Ioxcv12 commented Jul 13, 2025

I've seen your suggestions, so I've readded Crtl + P so that the user doesnt get confused when it doesnt work. Scatterer has the same issue with Alt + F11 and Alt + F10, so Scatterer supports both, and I think that is the best approach.

@R-T-B
Copy link
Member

R-T-B commented Jul 13, 2025

I've got some time coming up, will look at this and most likely merge it later this week.

Thank you for your patience.

@StollD
Copy link
Member

StollD commented Jul 13, 2025

I've seen your suggestions, so I've readded Crtl + P so that the user doesnt get confused when it doesnt work. Scatterer has the same issue with Alt + F11 and Alt + F10, so Scatterer supports both, and I think that is the best approach.

But if you re-add Ctrl + P as is, you still can't open the Parallax GUI without also opening the Kittopia GUI. Hence my suggestion to make it open a one-time notice and then subsequently ignore it.

@R-T-B
Copy link
Member

R-T-B commented Jul 13, 2025

Oh, I thought that's what he did? Oh well, when I get around to this I can add that if need be. Or you can edit the PR, either way.

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.

3 participants