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

imageadd: literally injector sprite for DNA injector. #3729

Merged
merged 2 commits into from
Oct 25, 2023
Merged

imageadd: literally injector sprite for DNA injector. #3729

merged 2 commits into from
Oct 25, 2023

Conversation

Daeberdir
Copy link
Member

@Daeberdir Daeberdir commented Oct 17, 2023 •
edited
Loading

Описание

Убирает древний, продублированный с простого шприца, спрайт ДНК инжектора и добавляет ДНК инжектору спрайты, схожие с обычными инжекторами - логичный визуал к его особенностям применения и названию. Инжекторы, копирующие все гены из буфера, будут иметь большой спрайт, пока инжекторы с 1 определённым блоком будут иметь спрайт "ручки". Теперь скиллишуй генетики не будут ошибочно просить снимать шлем рига или отказывать унати в выдаче генов. Спрайты предоставлены мисье @PiotrTheTchaikowsky, им же одобрены изменения в подобном виде.

Обьяснение Ведущего Спрайтера на тему этого недочёта

изображение

Демонстрация изменений

image image

@github-actions github-actions bot added the DMI Affects DMI files label Oct 17, 2023
@PiotrTheTchaikowsky
Copy link
Contributor

PiotrTheTchaikowsky commented Oct 18, 2023 •
edited
Loading

Можно было попросить сделать спрайт, чисто для инжекторов этих, а то теперь мы вместо спрайта шприца продублировали спрайт инжектора :|
Попробую что нибудь накидать

Я считаю спрайт ДНК инжектора уже устоявшимся в игре и как то нет какой либо причины его менять

@Daeberdir
Copy link
Member Author

С мобильного гита в драфт не перевести, так что з сего момента считайте, что оно в драфте, пока я не поменяю спрайт на спрайт мисье сверху.

@Honker9999999
Copy link
Contributor

Honker9999999 commented Oct 19, 2023 •
edited
Loading

С мобильного гита в драфт не перевести, так что з сего момента считайте, что оно в драфте, пока я не поменяю спрайт на спрайт мисье сверху.

Screenshot_2023-10-19-08-56-10-49_4641ebc0df1485bf6b47ebd018b5ee76

Снизу страницы, сюда жмякай.
Только у тебя будет "Convert to draft"

@Daeberdir
Copy link
Member Author

С мобильного гита в драфт не перевести, так что з сего момента считайте, что оно в драфте, пока я не поменяю спрайт на спрайт мисье сверху.

Screenshot_2023-10-19-08-56-10-49_4641ebc0df1485bf6b47ebd018b5ee76

Снизу страницы, сюда жмякай.
Только у тебя будет "Convert to draft"

Screenshot_20231019-085918_GitHub.jpg

Вот это мобильный гит. Тут ничего нету.

@Honker9999999
Copy link
Contributor

А, ты прям с приложения на мобилу? Тогда хз.
Я со странички (#3729) в браузере, там есть.

@Daeberdir Daeberdir marked this pull request as draft October 19, 2023 07:49
@Daeberdir Daeberdir changed the title bigfix: wrong sprite on DNA injector. imageadd: literally injector sprite for DNA injector. Oct 19, 2023
@Daeberdir Daeberdir marked this pull request as ready for review October 19, 2023 09:35
@Dimach Dimach merged commit 8f67f60 into ss220-space:master220 Oct 25, 2023
11 checks passed
@Daeberdir Daeberdir deleted the injector-always-has-been branch October 25, 2023 13:39
Etrnlmelancholy pushed a commit to Etrnlmelancholy/Paradise-SS220 that referenced this pull request Jan 3, 2024
@Daeberdir @Etrnlmelancholy
imageadd: literally injector sprite for DNA injector. (ss220-space#3729)
291433f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
DMI Affects DMI files
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Daeberdir @PiotrTheTchaikowsky @Honker9999999 @Dimach