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

Fix: Invicible posters #758

Merged
merged 2 commits into from
Nov 25, 2023
Merged

Fix: Invicible posters #758

merged 2 commits into from
Nov 25, 2023

Conversation

AyIong
Copy link
Collaborator

@AyIong AyIong commented Nov 24, 2023

Что этот PR делает

Чинит невидимые модульные постеры, довольно радикальным методом но да ладно...
Этот прок не трогался 6 лет, вряд ли тронется в ближайшем будущем.

Почему это хорошо для игры

Постеры видно

Изображения изменений

Не дам

Тестирование

Да

Changelog

🆑
fix: Невидимые постеры теперь видимые
/:cl:

@github-actions github-actions bot added the 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! label Nov 24, 2023
@PhantornRU
Copy link
Collaborator

Иу. Либо на ОФФы, либо в модуль.

@AyIong AyIong marked this pull request as draft November 24, 2023 10:37
@AyIong AyIong marked this pull request as ready for review November 24, 2023 12:21
@dj-34
Copy link
Collaborator

dj-34 commented Nov 24, 2023

а почему такая реализация то?

@dj-34
Copy link
Collaborator

dj-34 commented Nov 24, 2023

достаточно было назначить своё название для модульных постеров и перезаписать icon_state и icon в нужном

@AyIong
Copy link
Collaborator Author

AyIong commented Nov 24, 2023

достаточно было назначить своё название для модульных постеров и перезаписать icon_state и icon в нужном

Прок рандомизации, всё равно берёт иконку родителя, так что вариантов крайне мало, и это лучший

@AyIong AyIong merged commit f77bb73 into ss220club:master Nov 25, 2023
14 checks passed
@AyIong AyIong deleted the fix-posters branch November 25, 2023 10:44
@dj-34
Copy link
Collaborator

dj-34 commented Nov 26, 2023

достаточно было назначить своё название для модульных постеров и перезаписать icon_state и icon в нужном

Прок рандомизации, всё равно берёт иконку родителя, так что вариантов крайне мало, и это лучший

почему не перезаписать прок просто?

@dj-34
Copy link
Collaborator

dj-34 commented Nov 26, 2023

Ты накинул свой файл постеров, исправляя некоторые кастомные, но подвергаешь риску новые постеры, которые могут быть добавлены оффами 🤷

@AyIong
Copy link
Collaborator Author

AyIong commented Nov 26, 2023

Ты накинул свой файл постеров, исправляя некоторые кастомные, но подвергаешь риску новые постеры, которые могут быть добавлены оффами 🤷

Новые постеры добавляются редко, и мы их точно заметим рано или поздно
Поздно если в виде заглушки no name

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants