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

Big UI commit - GUINode, MaskedFlag, PieChart, etc #169

Merged
merged 1 commit into from
Dec 11, 2023
Merged

Conversation

Hop311
Copy link
Contributor

@Hop311 Hop311 commented Dec 7, 2023

  • Updated sim submodule to include a lot of recent work (and some bug fixes, e.g. the MSVC v19.38 lexy bug)
  • Added GFXMaskedFlagTexture and GFXPieChartTexture
  • Added GUINode to handle generating and controlling UI defined in .gui and .gfx files (UIAdapter still does the actual generation, GUINode just provides the GDScript/godot-cpp interface)
  • Changed ProvinceOverviewPanel and Topbar to use GUINode, removing old GameSpeedPanel
  • Fixed some issues with the LoadingScreen thread (trying to wait for itself to finish)
  • Added lots of functionality to the ProvinceOverviewPanel, including:
    • Proper pop icons for slave state, admin efficiency and owner %age
    • Terrain image updates with terrain type
    • working pie charts for pop type, ideology and culture (ideology doesn't show anything because it's not generated in the SIM, but the pipeline from there to the UI works)
    • Crime icon and name reflects province's crime modifier
    • liferating bar fixed
    • colony status accurately shown (although only at province level, needs more state work to be shown in all provinces in a state)
    • controller country flag icon
    • loads of other labels, some with placeholder text, others with correct data
  • Added dedicated script for Topbar, including player flag and name control and working date and speed/pause buttons

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants