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

Turfs after the first one on a tile are ignored #37

Closed
DamianX opened this issue May 14, 2020 · 2 comments
Closed

Turfs after the first one on a tile are ignored #37

DamianX opened this issue May 14, 2020 · 2 comments
Labels
enhancement New feature or request wontfix This will not be worked on

Comments

@DamianX
Copy link

DamianX commented May 14, 2020

image

image

DM supports having more than one /turf on a tile.
If I recall correctly, turfs after the first are added as images to the first one. Or it could be the other way around. I don't remember.
/vg/station uses this feature to make shuttle corners with space underneath them, but we're probably stupid.

Is this something you would want to support?

@SpaiR
Copy link
Owner

SpaiR commented May 14, 2020

Technically speaking, BYOND doesn't support multiple turfs per tile.

http://www.byond.com/docs/ref/#/turf

When you create a new turf with new(), it always replaces the old one.

DM map editor do has an undocumented behavior when it allows multiple turfs. In game they are merged into one. But I found this unintuitive and inconsistent with the documentation. So I've made rule 'one turf per tile' on purpose.

Other codebases (TG and all downstreams) have their shuttle walls implemented as objects, not turfs. I would recommend to do the same.

@SpaiR SpaiR closed this as completed May 14, 2020
@SpaiR SpaiR added wontfix This will not be worked on enhancement New feature or request labels May 14, 2020
@DamianX
Copy link
Author

DamianX commented May 14, 2020

Understandable, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants