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

Basic GUI Variables set up and piped for Politics Menu #377

Merged
merged 1 commit into from
Feb 8, 2025

Conversation

apassanta
Copy link
Contributor

@apassanta apassanta commented Feb 6, 2025

Added all the basic GUI variables and got them piped from the .gui file. Also includes placeholder functions for all added buttons.

Note:
This does not include more complex listbox/panel entries such as
-Ideology Option
-Issue Option
-Reform Option
-Decision Entry
-Rebel Entry
-Party Window
-Select New Party
-Party Issue Option
-Release Nation Entry
-Movement Entry

I didn't know how to handle these, some of these may be better served being sub classes, especially those that are listbox entries.

Also didn't have time to test building it so if it breaks, oops

@apassanta apassanta requested a review from Hop311 February 6, 2025 06:15
Copy link
Contributor

@Hop311 Hop311 left a comment

Choose a reason for hiding this comment

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

Nice work, exactly how I'd go about starting a menu 👍
There are just a few places where function variants have been switched around, try running the program locally and fixing any errors that come up

Copy link
Contributor

@Hop311 Hop311 left a comment

Choose a reason for hiding this comment

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

Final tweaks, I've tested the PR with these changes and it'll be ready to merge, just need to squash it down to a single commit first.

Fixed piping functions

Update game/src/Game/GameSession/NationManagementScreen/PoliticsMenu.gd

Co-authored-by: Hop311 <[email protected]>

Update game/src/Game/GameSession/NationManagementScreen/PoliticsMenu.gd

Co-authored-by: Hop311 <[email protected]>
@apassanta apassanta merged commit c5a500a into OpenVicProject:master Feb 8, 2025
9 checks passed
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