Skip to content
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

Fixes for new brig #5217

Merged
merged 10 commits into from
Dec 16, 2023
Merged

Fixes for new brig #5217

merged 10 commits into from
Dec 16, 2023

Conversation

spartanbobby
Copy link
Contributor

@spartanbobby spartanbobby commented Dec 14, 2023

About the pull request

Fixes stuff about the new brig

Lights on windows
Lack of warning stripes under bulkheads
Lack of warning stripes under some doors
Fixes up some of the tiling to look better (hard to describe in the PR nanu knows about it)
Removes RNG loot spawner that could spawn thermals in the brig
Changes some 1x1 area
Changes the location of some of the semiotics
Makes the north and south brig maint area symmetrical again
Fixes layering issues
Rewalls the CMPs office so that the outer brig hallways are symmetrical
Changes Brig Phone categories to "MP Dept."
fixes incorrect brig lockers
fixes incorrect door and button IDs
fixes incorrect door names
fixes incorrect pipe placement

Explain why it's good for the game

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑 SpartanBobby, LCMS1
maptweak: Numerous Fixes for new brig
/:cl:

@github-actions github-actions bot added the Mapping did you remember to save in tgm format? label Dec 14, 2023
.vscode/settings.json Outdated Show resolved Hide resolved
@BadAtThisGame302
Copy link
Contributor

Should probably fix that lil corner there too, it's one down.

image

@realforest2001
Copy link
Member

realforest2001 commented Dec 15, 2023

Not sure if you picked it up, but the warden's office phone is sharing the name of the CMPs office phone
Edit: While at it, may be worth moving the three brig related phones (Warden/CMP/Main) to their own category.

Copy link
Member

@Nanu308 Nanu308 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixes we need

@Nanu308 Nanu308 added the Mapping Approved adds 500 new dict keys label Dec 16, 2023
@Nanu308 Nanu308 added this pull request to the merge queue Dec 16, 2023
Merged via the queue into cmss13-devs:master with commit 872c20c Dec 16, 2023
26 checks passed
cm13-github added a commit that referenced this pull request Dec 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mapping Approved adds 500 new dict keys Mapping did you remember to save in tgm format?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants