Skip to content

Add F5/F9 save and load shortcuts and fix graphics timing at 60 FPS - #16

Open
slmkhanahmed wants to merge 1 commit into
SaturninTheAlien:mainfrom
slmkhanahmed:main
Open

Add F5/F9 save and load shortcuts and fix graphics timing at 60 FPS#16
slmkhanahmed wants to merge 1 commit into
SaturninTheAlien:mainfrom
slmkhanahmed:main

Conversation

@slmkhanahmed

@slmkhanahmed slmkhanahmed commented Jul 13, 2026

Copy link
Copy Markdown

Summary

This pull request adds:

  • F5 as a save shortcut
  • F9 as a load shortcut
  • A graphics fix for gameplay at 60 FPS

Changes

  • Added handling for the F5 save key
  • Added handling for the F9 load key
  • Updated graphics-related timing or rendering behavior for 60 FPS

Testing

  • Confirmed that F5 saves the game
  • Confirmed that F9 loads the game
  • Tested gameplay at 60 FPS
  • Confirmed that existing save/load controls still work

Notes

I have reviewed the submitted code and understand how it works.

@slmkhanahmed
slmkhanahmed marked this pull request as draft July 13, 2026 07:31
@slmkhanahmed
slmkhanahmed marked this pull request as ready for review July 13, 2026 07:33
@slmkhanahmed

slmkhanahmed commented Jul 13, 2026

Copy link
Copy Markdown
Author

The pull request is ready for review. GitHub shows “Cannot update this protected ref,” so a repository maintainer will need to review and merge it.

@slmkhanahmed slmkhanahmed changed the title add f5 save with f9 load fix graphic with 60fps Add F5 save and F9 load shortcuts and fix 60 FPS graphics Jul 13, 2026
@slmkhanahmed slmkhanahmed changed the title Add F5 save and F9 load shortcuts and fix 60 FPS graphics Add F5/F9 save and load shortcuts and fix graphics timing at 60 FPS Jul 13, 2026
@SaturninTheAlien

Copy link
Copy Markdown
Owner

Thank you very much for your contribution to Pekka Kana 2!
Before I merge this PR for testing, could you please remove .runtime_test_data from the commit and add it to .gitignore?

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