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

List small wonders in the client, but hide them by default #1305

Merged
merged 3 commits into from
Aug 23, 2022

Conversation

lmoureaux
Copy link
Contributor

The client has always known which small wonders any player had built (#318). This adds code to show this in the client. For backward compatibility, the patch also hides small wonders by default, keeping only great wonders. This can be controlled using a new effect Wonder_Visible. All shipped rulesets use the old way.

Closes #318.

This shows information that has always been available, but was never shown.

Similar to Freeciv osdn#883354.
Closes longturn#318.
This effect controls whether information about small wonders is visible to
players (and hence sent to the client). It lets one control it for each wonder
type separately, or possibly make it depend on the government (it is sadly not
possible to make it depend on the diplomatic state).

See longturn#318.
@lmoureaux lmoureaux requested a review from jwrober August 21, 2022 23:51
@lmoureaux
Copy link
Contributor Author

Figured I'd make some more changes here, hold on

This sounds like something people may want (see wonders only if you have
embassy or something like that), so make it possible to use DiplRel
requirements with the Wonder_Visible effect.
@lmoureaux
Copy link
Contributor Author

updated

Copy link
Collaborator

@jwrober jwrober left a comment

Choose a reason for hiding this comment

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

Looks great in my local test of a saved game.

@jwrober jwrober merged commit 29c5ac1 into longturn:master Aug 23, 2022
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.

Everyone can see everyone's small wonders
2 participants