maniator / maniator/verticopolis

[P3] rental-housekeeping-overlay-blank: rentals get no cleanliness cell, reading like an unreachable hotel room

Open Beginner friendly
#679 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

good first issue
Dominant language
TypeScript
Stars
5
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Deferred from the /gds-code-review Acceptance Auditor on PR #633 (Modern rental living, v2.0.0).

src/engine/sim/congestion.ts:276 gives u.kind === "condo" an explicit tint: "na" cell, with a comment recording exactly why: "a blank condo reads exactly like an uncovered hotel room". That is a deliberate fix for a real ambiguity in the housekeeping overlay.

The two new residential kinds are labor-free by design (GDD §7, D9) but get no cell at all, so a Studio or Apartment floor on the cleanliness map is indistinguishable from a hotel room no housekeeping crew can reach. This is the same ambiguity the condo "na" tint exists to prevent, reintroduced for the new kinds.

The occupancy and satisfaction overlays are fine: they are kind-generic (FACILITIES[u.kind].population > 0).

Fix when picked up: give both rental kinds the same tint: "na" treatment as the condo.

/gds-code-review.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in src/engine/sim/congestion.ts at line 276 and read the existing condo tint handling and its rationale. Apply the same tint: "na" treatment to the two rental kinds, then verify that Studio and Apartment cells are no longer blank on the cleanliness overlay while occupancy and satisfaction overlays remain unchanged.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
game-dev
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
84/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.