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 saving zoom mode #1482

Merged
merged 3 commits into from
Sep 23, 2024
Merged

Fix saving zoom mode #1482

merged 3 commits into from
Sep 23, 2024

Conversation

AyIong
Copy link
Collaborator

@AyIong AyIong commented Sep 8, 2024

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

В названии всё сказано
Плюс выбранный режим теперь сохраняется

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

Больше никаких искривлённых спрайтов и рябящих пикселей

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

Ну зашёл, ну тыкнул на билинейку, вышел, перезашёл, билинейка сохранилась, нанесённый глазам урон непоправим

Changelog

🆑
fix: Выбранный вручную режим масштабирования, теперь должен сохраняться. Чтобы наверняка: После выбора режима, закройте окно игры нажав на крестик справа сверху.
/:cl:

@dj-34
Copy link
Collaborator

dj-34 commented Sep 8, 2024

опять мыло смотреть?

@AyIong
Copy link
Collaborator Author

AyIong commented Sep 8, 2024

опять мыло смотреть?

Чё сразу мыло?
Плюс спрайты не искажает :aga:

@dj-34
Copy link
Collaborator

dj-34 commented Sep 10, 2024

ну и что с тобой делать? сломать колени, дать леща?

@AyIong
Copy link
Collaborator Author

AyIong commented Sep 10, 2024

ну и что с тобой делать? сломать колени, дать леща?

Давай всё сразу

@dj-34
Copy link
Collaborator

dj-34 commented Sep 23, 2024

Ну я хз в общем, мне кажется наши игроки уже привыкли к crispy image, из-за чего изменение будет конкретным даунгрейдом. Я не уверен что у нас много игроков обращают на то, у кого там глаз потёк, или лишний пиксель появился.

@AyIong AyIong changed the title Change default zoom mode to point sample Fix saving zoom mode Sep 23, 2024
@dj-34 dj-34 merged commit 0e4bfee into ss220club:master Sep 23, 2024
18 checks passed
@AyIong AyIong deleted the less-artifacts branch September 23, 2024 15:49
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