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

When entering a tile check to see if we've met a new civ #573

Open
wants to merge 6 commits into
base: Development
Choose a base branch
from

Conversation

TomWerner
Copy link
Contributor

We can now discover new civs in a pure C7 game!

#571

@TomWerner TomWerner force-pushed the twrner/discover-new-civs branch from c2dd9f7 to 3c6311a Compare February 21, 2025 01:39
This is primarily based on the english translation of this german wiki: https://wiki.civforum.de/wiki/Stadtfeldertrag_(Civ3)

#541
We can now see the result of moving citizens around.

#541
For the city scene, where we show the name, what's produced, and how many turns are left until production/growth, we just grab information from the game data when drawn. We can do that for the lower right info box as well, ensuring that things are up to day whenever something happens, like a road being finished or a citizen being moved to a different tile.

This allows us to get rid of some unneeded event logic.
We now have the lower right hand "D" button for opening the diplomacy popup, and the popup lists the civs we know and whether we're at war with them.

In the LEAD struct I changed some byte arrays to longs/ints to make it easier to dump them to the console when debugging.

In order for the "D" button to work I had to shuffle the "GameStatus" node in front of the unit buttons node, because the invisible HBox for the unit buttons was intercepting all the mouse events.

#571
We can now discover new civs in a pure C7 game!

#571
@TomWerner TomWerner force-pushed the twrner/discover-new-civs branch from 3c6311a to de025f3 Compare February 21, 2025 14:51
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.

1 participant