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.

Sign up for GitHub

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

Jump to bottom

add 8 new screenhead faces #15773

Merged
merged 2 commits into from Sep 13, 2023
Merged

add 8 new screenhead faces #15773

merged 2 commits into from Sep 13, 2023

Conversation

Retrino
Copy link
Contributor

@Retrino Retrino commented Sep 9, 2023

About the PR

as title says, adds 8 faces (animated and unanimated) to screenhead borgs. examples below

example.mov

Why's this needed?

screenheads didnt have very many symbol/non face-ish options, which 5 of the 8 of these are

Changelog 

(u)Retrino
(+)Added 8 new faces to cyborg screenheads. Sly, Elated, Blush, Battery, Error, Loading, Pong, and Hypnotized

@boring-cyborg boring-cyborg bot added the C-Sprites Automatically applied on any .dmi or icons folder change label Sep 9, 2023
@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 9, 2023
frawhst
frawhst requested changes Sep 12, 2023
View reviewed changes
Copy link
Member

@frawhst frawhst left a comment

Choose a reason for hiding this comment

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

You re-added some gib sprites that were removed in a recent thing, should cut those out again
image

(sprites are good though)

@Retrino
Copy link
Contributor Author

Retrino commented Sep 12, 2023

You re-added some gib sprites

should be fixed now

@frawhst frawhst merged commit 88b0e9e into goonstation:master Sep 13, 2023
21 checks passed
github-actions bot pushed a commit that referenced this pull request Sep 13, 2023
@robuddybot
Changelog for #15773 [skip ci]
3502e75
@Retrino Retrino deleted the screenheads-2 branch April 19, 2024 05:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Azrun Azrun Azrun approved these changes

@frawhst frawhst Awaiting requested review from frawhst

Assignees
No one assigned
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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Retrino @Azrun @frawhst