Skip to content

Add a base overview to the info screen#196

Merged
nthykier merged 1 commit intomasterfrom
testing/base-display
Feb 15, 2020
Merged

Add a base overview to the info screen#196
nthykier merged 1 commit intomasterfrom
testing/base-display

Conversation

@nthykier
Copy link
Member

Give the player a quick insight into their bases and their status.

We show 3 numbers for the player:

  • The number of active bases (that can sustain the singularity)
  • The total number of bases (all inclusive)
  • In brackets, the number of bases not able to maintain the
    singularity and also not under construction/building items.

The display has color codes to make the player aware of the following
conditions (in order):

  • Only one active base left (default "red")
  • Idle bases that cannot sustain the singularity (default "yellow")
  • Too many bases in total (default "orange")
  • Otherwise everything ok (default "white")

Signed-off-by: Niels Thykier [email protected]

@nthykier
Copy link
Member Author

Screenshots of it in action:

Here it is with 2 bases both of them active
base-display-simple

Here is an instance of two bases active, 1 under construction and one inactive base (idle and not able to sustain the player)

base-display-one-idle-one-under-construction

And if we remove one of the active bases:
base-display-one-active-one-idle-one-under-construction

@Quix0r
Copy link
Contributor

Quix0r commented Dec 11, 2019

I think this patch exposes how it should be played, having 3 bases (small warehouses) distributed well.

@nthykier
Copy link
Member Author

I think this patch exposes how it should be played, having 3 bases (small warehouses) distributed well.

Thanks for the feedback. Could you clarify whether you think this is an improvement ? I can read our comment both as bad ("it exposes something it should not") or good ("it guides the player to avoid issues").

Give the player a quick insight into their bases and their status.

We show 3 numbers for the player:

 * The number of active bases (that can sustain the singularity)
 * The total number of bases (all inclusive)
 * In brackets, the number of bases *not* able to maintain the
   singularity and also *not* under construction/building items.

The display has color codes to make the player aware of the following
conditions (in order):

 * Only one active base left (default "red")
 * Idle bases that cannot sustain the singularity (default "yellow")
 * Too many bases in total (default "orange")
 * Otherwise everything ok (default "white")

Signed-off-by: Niels Thykier <[email protected]>
@nthykier nthykier force-pushed the testing/base-display branch from 7a92e00 to c7ad6af Compare February 15, 2020 14:12
@codecov-io
Copy link

Codecov Report

Merging #196 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #196   +/-   ##
=======================================
  Coverage   47.41%   47.41%           
=======================================
  Files          41       41           
  Lines        5534     5534           
=======================================
  Hits         2624     2624           
  Misses       2910     2910

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0c38c4c...c7ad6af. Read the comment docs.

@nthykier nthykier merged commit c7ad6af into master Feb 15, 2020
@nthykier nthykier deleted the testing/base-display branch February 15, 2020 14:15
@nthykier nthykier mentioned this pull request Feb 15, 2020
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.

3 participants