Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Improvements to keybinds #2459

Draft
wants to merge 34 commits into
base: development
Choose a base branch
from

Conversation

j-gynn
Copy link
Contributor

@j-gynn j-gynn commented Jun 14, 2024

About The Pull Request

Hi. I'm back at it again. Today, I'd like to share the following keybind things:

New Keybinds

  • Any screen in the main game - 1-8 to jump to specific screen ("events screen", "camp screen", "list screen", "patrol screen", "leader den screen", "med den screen", "clearing screen", "warrior den screen")
  • ESC key now mostly functions like the back button in most pages (WIP)
    • When on Camp, List or Patrol screens, ESC brings you to Events
    • If on Events, ESC opens the "quit game?" dialogue. Pressing ESC again closes that dialog without saving or quitting.
  • ListScreen -ESC or ENTER to deselect search box
  • ListScreen - TAB or ENTER to select the search box

The implementation is partial at the moment as I know scribble is going through and doing some UI work and I don't wanna cause problems for them. I just got Inspired after spotting that bug and yeah, whoops. I would like to see if it's possible to make the entire game controllable via keyboard by tabbing through things & having highlight modes etc but that's kind of far in the future. I just wanted to work on something this afternoon and here we are.

Why This Is Good For ClanGen

Giving some love to the keybind users. I think the ListScreen changes should ignore the keybind toggle - they currently don't, but I think it makes sense.

Linked Issues

Fixes #2456

Proof of Testing

Link to the Discord testing thread

Changelog/Credits

j-gynn and others added 30 commits June 14, 2024 11:57
…guard against it being impossible to give an injury)
# Conflicts:
#	scripts/screens/ClanScreen.py
#	scripts/screens/ListScreen.py
#	scripts/screens/PatrolScreen.py
…cc_Fix

[Bugfix] fixed accessory ceremonies
…-util-log

Bugfix: Adds default relationship log text
…-injuries-fix

[BUGFIX]: patrol only tries to award injuries that the cat doesn't already have
…borked

Fix accessory generation for accessory ShortEvents
…nvited-kit

Bugfix: existing kit in clan is not considered new when parent reinvited
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.

[BUG] Keybinds crash the game
4 participants