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

Big moody light update #37296

Open
wants to merge 60 commits into
base: Bleeding-Edge
Choose a base branch
from

Conversation

SECBATON-GRIFFON
Copy link
Contributor

@SECBATON-GRIFFON SECBATON-GRIFFON commented Oct 22, 2024

[content][sprites]

What this does

Adds moody lights to the following:

  • Vents, scrubbers, pipe meters, do not display nice as floor stuff with things over it, so just heaters and freezers
  • Canisters
  • SMES
  • Virology incubators, centrifuges and disease analysers
  • The nuke
  • Light tubes and bulbs
  • Microwaves
  • Crates and lockers (also does a lot of duplicate code and sprite cutdown as a lot of stuff was converted to overlays in the process of adding this)

image
image
image
image

Closes #36992.
Refactors the proc for adding moody lights so that it doesn't need to pass an icon file, instead checks one defined on the atom. Icon state has also now been made optional, with a choice to use one defined as a var on an atom or pass a custom one (defaults to the atom icon state if neither are provided)

Why it's good

lets people find more things in the dark, most importantly THE NUKE

How it was tested

viewing the items in game and interacting with them, including emagging crates and lockers

Changelog

🆑

  • rscadd: Heaters, freezers, canisters, SMES units, the other virology machines, the nuke, light tubes, light bulbs, microwaves, crates and lockers now have moody lighting!

@D3athrow-Issues D3athrow-Issues added Content Adds something. Neat! Sprites Pretty colors. labels Oct 22, 2024
@angryturnip
Copy link
Contributor

Could you show images/video of what these will look like so people can vote based on how they look?

@SECBATON-GRIFFON
Copy link
Contributor Author

Could you show images/video of what these will look like so people can vote based on how they look?

working on that right now

@angryturnip
Copy link
Contributor

Also, for the floor stuff, the lights would appear over player sprites if someone's walking over them wouldn't they?

@SECBATON-GRIFFON
Copy link
Contributor Author

SECBATON-GRIFFON commented Oct 22, 2024

Also, for the floor stuff, the lights would appear over player sprites if someone's walking over them wouldn't they?

that's the way it is for everything moody light related, (just the lighting itself, not the actual sprites) i was working on a PR to fix this effect but hit a roadblock and got no results. if it's necessary i could try do it in this one

@DeityLink
Copy link
Collaborator

DeityLink commented Oct 22, 2024

there's no fix for that. and that's why I've mostly avoided giving moody lights to floor objects, bare very few exceptions.

they very concept of moody lights is that an additive overlay is put on top of everything. You can't "cover" that light. It's the illusion of making things seem like they emit light, when in reality it's more akin to shining a flashlight above a piece of paper.

to "fix" that, you'd have to invent something else entirely, that wouldn't be "moody lights" anymore.

@SECBATON-GRIFFON
Copy link
Contributor Author

SECBATON-GRIFFON commented Oct 22, 2024

there's no fix for that. and that's why I've mostly avoided giving moody lights to floor objects, bare very few exceptions.

they very concept of moody lights is that an additive overlay is put on top of everything. You can't "cover" that light. It's the illusion of making things seem like they emit light, when in reality it's more akin to shining a flashlight above a piece of paper.

to "fix" that, you'd have to invent something else entirely, that wouldn't be "moody lights" anymore.

commented out the floor stuff for now since i suck at working with blend modes to mask things

@SECBATON-GRIFFON SECBATON-GRIFFON marked this pull request as ready for review October 23, 2024 00:08
@SECBATON-GRIFFON SECBATON-GRIFFON changed the title [MDB IGNORE] Big moody light update Big moody light update Oct 23, 2024
@SECBATON-GRIFFON SECBATON-GRIFFON marked this pull request as draft October 31, 2024 19:50
@SECBATON-GRIFFON SECBATON-GRIFFON marked this pull request as ready for review October 31, 2024 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content Adds something. Neat! Sprites Pretty colors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Moody light todo list
4 participants