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

Map Wnd "Production" and "Imperial Stockpile" should show transfer limit #2030

Open
agrrr3 opened this Issue Mar 5, 2018 · 5 comments

Comments

Projects
None yet
4 participants
@agrrr3
Contributor

agrrr3 commented Mar 5, 2018

"Production" in the Map Windows shows

  • Used
  • Output
  • Target Output
  • Stockpile
  • Stockpile Used
  • Next Turn Stockpile

"Imperial Stockpile" in the Map Windows show

  • Stockpile
  • Stockpile Used
  • Next Turn Stockpile

They should also show the maximum amount which can be used per turn. Also the target value for the this maximum is an important value to show.

  • Stockpile
  • Stockpile Use Limit
  • Target Use Limit
  • Stockpile Used
  • Next Turn Stockpile

Note "Stockpile Use Limit", "Next Turn Stockpile" both have 19 chars

In the production window the term "Maximum Use" is used for the "Stockpile Use Limit". Maybe the "Production" and "Imperial Stockpile" should show a different set of values. Or use different stringtable entries...

@Dilvish-fo

This comment has been minimized.

Show comment
Hide comment
@Dilvish-fo

Dilvish-fo Mar 5, 2018

Member

They should also show the maximum amount which can be used per turn....In the production window the term "Maximum Use" is used for the "Stockpile Use Limit".

Isn't that Maximum Use currently displayed in the ProductionWnd main panel the maximum amount that can be used per turn? But I certainly agree that it should also be shown in the tooltip for the Imperial Stockpile.

Member

Dilvish-fo commented Mar 5, 2018

They should also show the maximum amount which can be used per turn....In the production window the term "Maximum Use" is used for the "Stockpile Use Limit".

Isn't that Maximum Use currently displayed in the ProductionWnd main panel the maximum amount that can be used per turn? But I certainly agree that it should also be shown in the tooltip for the Imperial Stockpile.

@geoffthemedio

This comment has been minimized.

Show comment
Hide comment
@geoffthemedio

geoffthemedio May 1, 2018

Member

Transfer limit was added to the stockpile tooltip in 845e2aa

Member

geoffthemedio commented May 1, 2018

Transfer limit was added to the stockpile tooltip in 845e2aa

@Vezzra

This comment has been minimized.

Show comment
Hide comment
@Vezzra

Vezzra May 13, 2018

Member

@geoffthemedio, @agrrr3 (and anyone else who might want to comment), I don't think this is absolutely necessary for the release, and propose to reassign it to opional.

Anyone any objections?

Member

Vezzra commented May 13, 2018

@geoffthemedio, @agrrr3 (and anyone else who might want to comment), I don't think this is absolutely necessary for the release, and propose to reassign it to opional.

Anyone any objections?

@geoffthemedio

This comment has been minimized.

Show comment
Hide comment
@geoffthemedio

geoffthemedio May 13, 2018

Member

I think the existing partial implementation is sufficient.

Member

geoffthemedio commented May 13, 2018

I think the existing partial implementation is sufficient.

@Vezzra

This comment has been minimized.

Show comment
Hide comment
@Vezzra

Vezzra May 14, 2018

Member

Ok, reassigning to optional.

Member

Vezzra commented May 14, 2018

Ok, reassigning to optional.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment