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

[MIRROR] Adds lighting height control (space color consistency) #215

Merged
merged 1 commit into from
Oct 23, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#24501
Original PR: tgstation/tgstation#79046

About The Pull Request

Adds support for modifying a light's "height"
You can think of this as the distance it is from the ground below it (Really it's the distance to the corners around it + 0.5 but yaknow) We use it to keep wall lights from looking weird, but well, not everything is a wall light

Floors tend to not be, and space in particular does not want to be treated as such.
In fact, it wants a NEGATIVE height, so it acts as if it in on top of all of its corners. This allows us to ensure that the starlight from space and the starlight from starlight overlays always have the same intensity and color, preventing weird lines from where the two intersect, or starlight feeling not very present in cases with only one turf

I've also bumped starlight's intensity form 0.75 to 1, this should help with the lines thing discussed above.

Why It's Good For The Game

image

Changelog

🆑 LemonInTheDark
add: Starlight should be a bit more intense, and flow better onto non space tiles
/:cl:

…IGNORE] (#24501)

* Adds lighting height control (space color consistency) (#79046)

## About The Pull Request

Adds support for modifying a light's "height"
You can think of this as the distance it is from the ground below it
(Really it's the distance to the corners around it + 0.5 but yaknow) We
use it to keep wall lights from looking weird, but well, not everything
is a wall light

Floors tend to not be, and space in particular does not want to be
treated as such.
In fact, it wants a NEGATIVE height, so it acts as if it in on top of
all of its corners. This allows us to ensure that the starlight from
space and the starlight from starlight overlays always have the same
intensity and color, preventing weird lines from where the two
intersect, or starlight feeling not very present in cases with only one
turf

I've also bumped starlight's intensity form 0.75 to 1, this should help
with the lines thing discussed above.

## Why It's Good For The Game

![image](https://github.com/tgstation/tgstation/assets/58055496/240d1b3f-52c8-4569-8e74-0d801cbdb84d)

## Changelog
:cl:
add: Starlight should be a bit more intense, and flow better onto non
space tiles
/:cl:

---------

Co-authored-by: Zephyr <12817816+ZephyrTFA@ users.noreply.github.com>
Co-authored-by: san7890 <the@ san7890.com>

* Adds lighting height control (space color consistency)

---------

Co-authored-by: LemonInTheDark <[email protected]>
Co-authored-by: Zephyr <12817816+ZephyrTFA@ users.noreply.github.com>
Co-authored-by: san7890 <the@ san7890.com>
@AnywayFarus AnywayFarus merged commit 0b38c68 into master Oct 23, 2023
23 checks passed
AnywayFarus added a commit that referenced this pull request Oct 23, 2023
@AnywayFarus AnywayFarus deleted the upstream-mirror-24501 branch October 23, 2023 08:06
Iajret pushed a commit that referenced this pull request Jan 3, 2024
…S?) (#215)

* the 17 tile range is insane

* makes that use a define
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants