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

Implement city management screen #20

Closed
SWY1985 opened this issue Sep 1, 2016 · 2 comments
Closed

Implement city management screen #20

SWY1985 opened this issue Sep 1, 2016 · 2 comments
Assignees
Labels
Milestone

Comments

@SWY1985
Copy link
Owner

SWY1985 commented Sep 1, 2016

No description provided.

@SWY1985 SWY1985 added the feature label Sep 1, 2016
@SWY1985 SWY1985 self-assigned this Sep 1, 2016
@SWY1985 SWY1985 added this to the 0.1.0-alpha1 milestone Sep 1, 2016
SWY1985 added a commit that referenced this issue Sep 5, 2016
SWY1985 added a commit that referenced this issue Sep 6, 2016
SWY1985 added a commit that referenced this issue Sep 7, 2016
SWY1985 added a commit that referenced this issue Sep 9, 2016
SWY1985 added a commit that referenced this issue Sep 9, 2016
@SWY1985
Copy link
Owner Author

SWY1985 commented Sep 10, 2016

Feedback by DracoNoir:
= City Management Screen:

  • There should be spaces between Happy, Content, Unhappy, Entertainers/Tax collectors/scientists (not between these 3) and citizens. Also entertainers in city size 1 (like you ever would!) also have a space first, then the entertainer.
  • Unable to change entertainers into taxmen or scientists. Clicking leaves management screen (WIP?)
  • Creating entertainers does not (yet) create luxuries or happy citizens.
  • Nearby cities are currently able to share map squares for resources.
  • City management screen does not show food deficits (black icons). Also shields. Not yet sure about trade. (deliberate for now?)
  • Food consumption maths is off. Should be 2n+1 where N is city size (at least on chieftain - yet to test other difficulties).
  • After 'buying' a unit, clicking change before leaving (or building the unit, ie while production window for the unit is full of shields) shows cheaper units as taking 0 or negative amounts of turns - this should read 1.
  • Clicking on the city square on resource minimap resets all entertainers and seems to prioritise shields over food. Thought you could do this in original game too (wanted to see if original prioritised food) but screen just blinked with no change? EDIT: Original game DOES reset resource fields when clicking an empty square when all citizens are workers. Original seemed to prioritise food for growth, leaving a shortage of shields for unit support under monarchy govt. when it was possible to have a balance and no shortages of food OR shields.

@iStruggle
Copy link

Please ignore my note on food consumption. My mistake. All my other notes have now been implemented, except luxuries and happy citizens/taxmen and scientists.

SWY1985 pushed a commit that referenced this issue Aug 8, 2018
SWY1985 pushed a commit that referenced this issue Aug 8, 2018
SWY1985 pushed a commit that referenced this issue Aug 8, 2018
SWY1985 pushed a commit that referenced this issue Aug 8, 2018
SWY1985 pushed a commit that referenced this issue Aug 8, 2018
SWY1985 pushed a commit that referenced this issue Aug 8, 2018
SWY1985 pushed a commit that referenced this issue Aug 8, 2018
m35 pushed a commit to m35/CivOne that referenced this issue Oct 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants