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

misc: reconfigure UI toggling mechanism #1683

Merged
merged 3 commits into from
Oct 7, 2024

Conversation

lahm86
Copy link
Collaborator

@lahm86 lahm86 commented Oct 7, 2024

No issue raised as this is all unreleased so far. As this renames the previous general UI option, your current setting will be reset effectively.

Checklist

  • I have read the coding conventions
  • I have added a changelog entry about what my pull request accomplishes, or it is an internal change

Description

This separates the general game UI state and photo mode UI state settings, and provides a mechanism for any future cases where specific UIs need to be togglable. There is perhaps a better place than the common UI module for the function, LMK what you think.

This will allow making specific UI hiding settings rather than a global
catch-all.
This makes a general UI hiding function available in libtrx, expecting
the relevant specific config setting to be provided.
For TR1, the call is moved away from shell.c into Game_ProcessInput.
This introduces a separate UI state setting for photo mode only.
@lahm86 lahm86 added Internal The invisible stuff Unreleased Applies to things that aren't in a stable release. TR1 labels Oct 7, 2024
@lahm86 lahm86 added this to the TR1X-4.5 milestone Oct 7, 2024
@lahm86 lahm86 self-assigned this Oct 7, 2024
@lahm86 lahm86 requested review from a team as code owners October 7, 2024 14:14
@lahm86 lahm86 requested review from rr-, walkawayy and aredfan and removed request for a team October 7, 2024 14:14
Copy link
Collaborator

@rr- rr- left a comment

Choose a reason for hiding this comment

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

This is really elegant and clever. Thanks for coming up with it – I'd never have thought of this solution myself.

@lahm86 lahm86 merged commit 53101ab into LostArtefacts:develop Oct 7, 2024
5 checks passed
@lahm86 lahm86 deleted the reconfigure-ui-toggle branch October 7, 2024 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internal The invisible stuff TR1 Unreleased Applies to things that aren't in a stable release.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants