Skip to content

No double backdrop for openspace#12523

Merged
hry-gh merged 2 commits into
cmss13-devs:masterfrom
Drulikar:no_double_backdrop
Jun 28, 2026
Merged

No double backdrop for openspace#12523
hry-gh merged 2 commits into
cmss13-devs:masterfrom
Drulikar:no_double_backdrop

Conversation

@Drulikar

@Drulikar Drulikar commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

About the pull request

This PR simply makes it where an openspace looking at an openspace does not add the openspace backdrop (causing it to be doubled+ up). I sort of want to also try just making it where openspaces don't even put other openspaces into vis contents, but then things moving around in that intermediate space are either going to have to update vis contents (which is expensive) or they're just be not rendered at all.

Visually I don't think there's a difference (but there likely would be for airborne things), so even better (see screenshots).

Explain why it's good for the game

Less unnecessary things being rendered, also may be the cause of some visual artifacts when at larger zoom levels.

Testing Photographs and Procedure

Screenshots & Videos

Before:
before
before2

image

After:
after
after2

image

Changelog

🆑 Drathek
fix: Openspace no longer doubles up the openspace backdrop when viewing an intermediate openspace
/:cl:

@Drulikar Drulikar added the Performance the game sucks and this makes it suck less label Jun 25, 2026
@github-project-automation github-project-automation Bot moved this to Awaiting Review in Review Backlog Jun 25, 2026
@cmss13-ci cmss13-ci Bot added Fix Fix one bug, make ten more size/XS Denotes a PR that changes 0-19 lines, ignoring generated files. labels Jun 25, 2026
@github-project-automation github-project-automation Bot moved this from Awaiting Review to Partially Approved / Mergeable in Review Backlog Jun 28, 2026
@hry-gh hry-gh added this pull request to the merge queue Jun 28, 2026
Merged via the queue into cmss13-devs:master with commit c7066dc Jun 28, 2026
28 checks passed
@github-project-automation github-project-automation Bot moved this from Partially Approved / Mergeable to Merged in Review Backlog Jun 28, 2026
cmss13-ci Bot added a commit that referenced this pull request Jun 28, 2026
@Drulikar Drulikar deleted the no_double_backdrop branch June 28, 2026 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix Fix one bug, make ten more Performance the game sucks and this makes it suck less size/XS Denotes a PR that changes 0-19 lines, ignoring generated files.

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

2 participants