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] Updates some outdated malf AI/bot ability icons #2881

Merged
merged 1 commit into from
Apr 14, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1975
Original PR: tgstation/tgstation#82614

About The Pull Request

I updated some action icons used by AIs and bots:

  • Every icon has been updated to the new sprites used here (where applicable)
    -- Redid some icons to look better hopefully
    -- Deleted some icons and changed their file paths to avoid duplicates
  • Replaced "voice_changer" with the same sprite for "ai_president" in mob/silicon/ai.dmi
  • Recolored "reactivate_cameras" to feature shading and a better green color

Sprites:

Old
image

New
image
(missing icons brought in from other files)

New gifs used
detonate_rcds override_machine

Why It's Good For The Game

I just wanted to do the intercom, but I saw some easily fixable outdated stuff so I freshened those up a bit too
And I thought the poor gas-mask AI face and green camera assembly looked awful, so I used another icon that fit for the former and gave the latter some better shades of green. Open to changing or undoing any of this of course!

Changelog

🆑 AMyriad
image: Updated the ability icons used by AIs and station bots
/:cl:

* Updates some outdated malf AI/bot ability icons (#82614)

## About The Pull Request
I updated some action icons used by AIs and bots:
- Every icon has been updated to the new sprites used here (where
applicable)
-- Redid some icons to look better hopefully
-- Deleted some icons and changed their file paths to avoid duplicates
- Replaced "voice_changer" with the same sprite for "ai_president" in
mob/silicon/ai.dmi
- Recolored "reactivate_cameras" to feature shading and a better green
color


### Sprites:
Old

![image](https://github.com/tgstation/tgstation/assets/143908044/2254e7f7-c7b5-4397-8470-ae8f1e11aa1b)

New

![image](https://github.com/tgstation/tgstation/assets/143908044/ba3a81fc-45b6-4564-bcc9-1701e34c565e)
(missing icons brought in from other files)

New gifs used

![detonate_rcds](https://github.com/tgstation/tgstation/assets/143908044/6580f63f-c459-4623-97f3-29c4c6f3d043)
![override_machine](https://github.com/tgstation/tgstation/assets/143908044/adbdc977-1b43-4d4b-a11e-40f120b095d0)



## Why It's Good For The Game
I just wanted to do the intercom, but I saw some easily fixable outdated
stuff so I freshened those up a bit too
And I thought the poor gas-mask AI face and green camera assembly looked
awful, so I used another icon that fit for the former and gave the
latter some better shades of green. Open to changing or undoing any of
this of course!

## Changelog
:cl:
image: Updated the ability icons used by AIs and station bots
/:cl:

* Updates some outdated malf AI/bot ability icons

---------

Co-authored-by: AMyriad <[email protected]>
@ReezeBL ReezeBL merged commit 80feb0d into master Apr 14, 2024
26 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-1975 branch April 14, 2024 19:07
AnywayFarus added a commit that referenced this pull request Apr 14, 2024
Iajret pushed a commit that referenced this pull request Jun 7, 2024
…2881)

* [NO GBP] Only add the armor plate prefix once. (#83650)

## About The Pull Request

Currently, this will add the prefix for every upgrade. We only want to
add it once.

## Why It's Good For The Game

Oops. This isn't working as intended.

## Changelog
:cl:
fix: The armor plate component only adds the prefix once.
/:cl:

* [NO GBP] Only add the armor plate prefix once.

---------

Co-authored-by: necromanceranne <[email protected]>
Co-authored-by: NovaBot13 <[email protected]>
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