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

Replace colored-tile decals with colored tiles. #17260

Merged

Conversation

warriorstar-orion
Copy link
Contributor

What Does This PR Do

This PR removes decals used for colored floors on the Lavaland Base, and replaces their floor turfs with the appropriate colored tiles.

I left the loading area, warning stripes, and other decals used in the Processing Area as those are still used on all station maps.

Why It's Good For The Game

This brings Lavaland Base more in line with current mapping practices. I assume there's also some negligible performance benefits to not rendering all these decals.

Images of changes

Before

Greenshot-16_06_21

After

Greenshot-13_18_27

Changelog

🆑
fix: Replace colored-tile decals with colored tiles on Lavaland Base.
/:cl:

Copy link
Member

@AffectedArc07 AffectedArc07 left a comment

Choose a reason for hiding this comment

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

These decals need to be put down

@S34NW
Copy link
Member

S34NW commented Dec 20, 2021

oh thank the lord for fixing that

Copy link
Member

@S34NW S34NW left a comment

Choose a reason for hiding this comment

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

mapping is fine, slight change in medbay is no issue (don't need a changelog BTW)

@dearmochi dearmochi added the Map Edit This PR will modify a map label Dec 20, 2021
@dearmochi dearmochi merged commit a929b23 into ParadiseSS13:master Dec 22, 2021
@warriorstar-orion warriorstar-orion deleted the mining_station_tiles branch January 26, 2024 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Map Edit This PR will modify a map
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants