in the ship lab remove the bottom research point list as it's redundant #222

New issue

Closed

opened 2026-09-05 19:37:32 +00:00 by metaheavens · 1 comment

metaheavens commented

2026-09-05 19:37:32 +00:00

Owner

No description provided.

metaheavens referenced this issue from a commit

2026-09-05 21:13:28 +00:00

Plan every craftable vessel, and stop restating the point cost

metaheavens commented

2026-09-05 21:13:45 +00:00

Author

Owner

Implemented in commit f775963 on main. Full suite: 166/166 passed.

Removed. Every upgrade card already carries its own RP have/need, and the detail block underneath was restating it for the selected card a few pixels lower - visible in the before/after: Heavy Slugs / +1 damage per shot. / Research points: 3/1 / Researching, where the card above it already said RP 3/1.

The points parameter went with it, along with the research_point_cost() lookup at the call site that existed only to feed it.

Implemented in commit `f775963` on `main`. Full suite: **166/166 passed**. Removed. Every upgrade card already carries its own `RP have/need`, and the detail block underneath was restating it for the selected card a few pixels lower - visible in the before/after: `Heavy Slugs / +1 damage per shot. / Research points: 3/1 / Researching`, where the card above it already said `RP 3/1`. The `points` parameter went with it, along with the `research_point_cost()` lookup at the call site that existed only to feed it.

metaheavens closed this issue

2026-09-05 21:13:46 +00:00

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

metaheavens/spaceship#222

No description provided.

Rows
Columns