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(again): Paintkit #571

Merged
merged 3 commits into from
Oct 29, 2023
Merged

FIX(again): Paintkit #571

merged 3 commits into from
Oct 29, 2023

Conversation

ErdGinalD
Copy link

@ErdGinalD ErdGinalD commented Oct 25, 2023

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

Переписал 1 строчку в проке для правильного отображения текстур. КЛЯНУСЬ ОНО РАБОТАЕТ ИДЕАЛЬНО

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

баги плоха

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

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

Запустил игру, она запустилась, выдохнул и выключил компьютер от греха подальше.

Copy link

@Furrior Furrior left a comment

Choose a reason for hiding this comment

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

И че это? Зачем?

@ErdGinalD
Copy link
Author

ErdGinalD commented Oct 26, 2023

И че это? Зачем?

Фиксит баг когда текстура слетала при -open значении прока go_out. Текстура -open бралась от самой первой, а не текущей. Больше ничего не меняет

@Furrior
Copy link

Furrior commented Oct 27, 2023

Рализация хуйня. Делай модульно

@ErdGinalD
Copy link
Author

Рализация хуйня. Делай модульно

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

@PhantornRU
Copy link
Collaborator

Переписал 1 строчку на +61 −0

@PhantornRU
Copy link
Collaborator

Вот теперь верю.

Copy link
Collaborator

@AyIong AyIong left a comment

Choose a reason for hiding this comment

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

Латна

@AyIong AyIong merged commit 1d84c40 into ss220club:master Oct 29, 2023
13 of 15 checks passed
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.

4 participants