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

Resprite critter gunbots #5923

Merged
3 commits merged into from Sep 19, 2021
Merged

Conversation

flappybatpal
Copy link
Contributor

@flappybatpal flappybatpal commented Sep 6, 2021

About the PR

Resprites gunbots with sprites by Matheus.
Adds glowing red eyes.
Adds animated treads.

Why's this needed?

They fit the new borgs better!

image

Changelog

(u)Flappybat and THISISANICEGAME 
(+)Resprited the critter security bot, now with animated treads and glowing eyes.

@boring-cyborg boring-cyborg bot added the C-Sprites Automatically applied on any .dmi or icons folder change label Sep 6, 2021
@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 6, 2021
@flappybatpal
Copy link
Contributor Author

Could not get the glowing eyes to work in the same way cyborg heads work. SELFILLUM is nearly the same but doesn't quite have the same change in different lighting. If anyone could work out the way to implement that it would be nice.

I copied the very simple implementation and sprites of Pali's PR and had 0 success.

@Flaborized
Copy link
Contributor

Looks good and fits resprite 👍

Comment on lines +21 to +25
New()
. = ..()
var/image/eye_light = image(icon, "mars_sec_bot_eye")
eye_light.plane = PLANE_SELFILLUM
src.UpdateOverlays(eye_light, "eye_light")
Copy link
Member

Choose a reason for hiding this comment

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

so you're saying this doesn't work?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh no that works, it's just pretty different to the implementation on borgs which use black pixels for eyes and put them on lighting plane.

eye_light.plane = PLANE_LIGHTING

@ghost ghost merged commit f60f6e4 into goonstation:master Sep 19, 2021
github-actions bot pushed a commit that referenced this pull request Sep 19, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Sprites Automatically applied on any .dmi or icons folder change size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants