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

Foot print fix #763

Merged
merged 14 commits into from
Dec 20, 2024
Merged

Foot print fix #763

merged 14 commits into from
Dec 20, 2024

Conversation

MalorMorfin
Copy link
Contributor

@MalorMorfin MalorMorfin commented Dec 5, 2024

Основные изменения

Теперь при попадании на кровь за маром будет идти след

Внешний вид следов после обновления
image

Как это улучшит игру

Крута

Ченджлог

:cl:
fix: Теперь следы стопы остаются на полу
/:cl:

@MalorMorfin
Copy link
Contributor Author

Вообще надо бы наверное прилегающие однобуквенные переменные переделать, но я подумаю...

@Helg2 Helg2 added the Fix Исправление или правка. label Dec 5, 2024
@Tatarla Tatarla requested review from homexp13 and Helg2 December 5, 2024 12:30
@Tatarla
Copy link
Collaborator

Tatarla commented Dec 6, 2024

Подправь цвет крови от гиба беносов, она сейчас оранжевая, а нужна зеленая/лаймовая

@MalorMorfin
Copy link
Contributor Author

Подправь цвет крови от гиба беносов, она сейчас оранжевая, а нужна зеленая/лаймовая

Ну там прикол что берется бейз колор, а он желтый, и если менять то сам цвет иконки поменяется, вот.. Можно брать блуд колор, но это выглядит как-то костыльно, у всего берется бейз колор, но только у ксен блуд... Но я чекну и изменю а дальше хельг пусть обосрет мой говнокод

@Helg2
Copy link
Collaborator

Helg2 commented Dec 6, 2024

Ну там прикол что берется бейз колор, а он желтый, и если менять то сам цвет иконки поменяется, вот.. Можно брать блуд колор, но это выглядит как-то костыльно, у всего берется бейз колор, но только у ксен блуд... Но я чекну и изменю а дальше хельг пусть обосрет мой говнокод

А почему просто не поменять цвет крови на зелёный? Не совсем уловил

@MalorMorfin
Copy link
Contributor Author

Ну там прикол что берется бейз колор, а он желтый, и если менять то сам цвет иконки поменяется, вот.. Можно брать блуд колор, но это выглядит как-то костыльно, у всего берется бейз колор, но только у ксен блуд... Но я чекну и изменю а дальше хельг пусть обосрет мой говнокод

А почему просто не поменять цвет крови на зелёный? Не совсем уловил

Кровь из ботинок берется с бейз колора того чего ты наступил, а не блуд колор, блуд колор в принципе не прописан, даже у крови, а бейз колор меняет цвет самого спрайта. Мне добовлять всему блуд колор и менять проверку что бы брался беййз колор только в случае отсутсвия блуд колор?

code/game/turfs/open.dm Outdated Show resolved Hide resolved
code/game/turfs/open.dm Outdated Show resolved Hide resolved
code/game/turfs/open.dm Outdated Show resolved Hide resolved
@MalorMorfin
Copy link
Contributor Author

Такой крутой код, у каждого второго предмета запрашивается цвет крови, который берется иногда не понятно откуда. А ну и человек прикол кто изначально писал код решил что если у всех следов базовой цвет будет красный - гений. Легче было переписать...

@MalorMorfin MalorMorfin requested a review from Helg2 December 13, 2024 03:52
@Helg2 Helg2 merged commit b133fe8 into PMC-Unga-Marines:master Dec 20, 2024
21 of 28 checks passed
github-actions bot added a commit that referenced this pull request Dec 20, 2024
@MalorMorfin MalorMorfin deleted the stpe_footprints branch December 23, 2024 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Исправление или правка.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants