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

Add an entry for the diary in the Action Menu #55045

Merged
merged 1 commit into from
Feb 5, 2022

Conversation

DeeUnderscore
Copy link
Contributor

Summary

Interface "The diary can now be accessed though the Action Menu, under Info"

Purpose of change

Make the diary accessible through the menu system, allowing for easier discoverability and access without binding a specific key

Describe the solution

Added the diary action to the "Info…" section of the Action Menu. Also, gave the action a friendly name: "Open Diary".

Describe alternatives you've considered

The item could also go under Misc section or even in the Main Menu, but I feel it makes sense under Info.

Testing

Built the game with the change and checked that the item is in the menu, and doesn't accidentally automatically receive a keybind when the game is launched with a blank profile.

Additional context

Screenshot of the Info section with the Open Diary option

@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Feb 3, 2022
@Maleclypse Maleclypse added the Info / User Interface Game - player communication, menus, etc. label Feb 4, 2022
@Rivet-the-Zombie Rivet-the-Zombie merged commit 7efd4f5 into CleverRaven:master Feb 5, 2022
@DeeUnderscore DeeUnderscore deleted the diary-menu branch March 15, 2022 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions Info / User Interface Game - player communication, menus, etc. json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants