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

Display fractional values for building time on the production tab #16179

Open
netnazgul opened this Issue Feb 8, 2019 · 3 comments

Comments

Projects
None yet
5 participants
@netnazgul
Copy link
Contributor

netnazgul commented Feb 8, 2019

Currently, the display value is rounded up to the nearest integer seconds count. So units with short build time (50, 45, 40, 35 ticks), while having different build time, will still all be displayed as "0:02". This also leads people to the confusion that multiple production buildings do not decrease actual production time as it stays the same in seconds which is the only value shown at the frontend.

Displaying fractional values up to 0.1 of a second might be considered.

@MustaphaTR

This comment has been minimized.

Copy link
Member

MustaphaTR commented Feb 8, 2019

Displaying fractional values up to 0.1 of a second might be considered.

I dunno if we have enough UI space for it, but if we show 1/100th or 1/60 of a second, we would be able to show the time accurately on normal speed, since 1 tick is 1/25th of a second on normal.

@penev92

This comment has been minimized.

Copy link
Member

penev92 commented Feb 8, 2019

I was actually never suggesting we show fractions in the timer that shows the remaining time - that's a timer and it's fine, also has no space.
I was only talking about the build tooltips, which show the total (presumably exact) build time, and only about showing tenths of a second.

@netnazgul netnazgul changed the title Display fractional values for building time left on the production tab Display fractional values for building time on the production tab Feb 8, 2019

@pchote

This comment has been minimized.

Copy link
Member

pchote commented Feb 8, 2019

👎 for adding unnecessary visual noise

@abcdefg30 abcdefg30 added the UI label Feb 8, 2019

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