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
Corporation fixes #139
Corporation fixes #139
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think I want the Great People lists to be visible in the civilopedia. Discovering all the names that exist should be part of the game, and a reward for getting those GPs, in my opinion.
Can you separate the actual bug fixes from this pull request so I can approve them on their own? Everything looks fine there for me.
Done |
@@ -102,6 +102,7 @@ def __init__(self): | |||
self.W_INFLUENCES_AREA = self.X_CITY2_AREA + self.W_CITY_AREA - self.X_INFLUENCES_AREA | |||
self.H_INFLUENCES_AREA = self.H_REQUIREMENTS_AREA | |||
self.INFLUENCES_ID = "InfluencessAreaWidget" | |||
self.iCorporationTech = iCorporation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest to solve this using
from Consts import *
from Consts import iCorporation as iCorporationTech
Should work iirc. At least the iCorporationTech shouldn't be a member variable of the CvCorporationScreen class.
Incorporated (pun intended) suggestions. |
Fixed wrong buttons in corporation advisor
Fixed corpotion pedia page when playing Brazil
REQUIRES RECOMPILING THE DLL