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

Zoom fixes #741

Conversation

definitelynotspaghetti
Copy link
Contributor

@definitelynotspaghetti definitelynotspaghetti commented Dec 3, 2024

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

Рефактор зума, префы для вайдскрина, без добавления ультравайдскрина

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

При зуме больше не будет зависать

Ченджлог

:cl:
refactor: Рефактор кода зума
/:cl:

@Tatarla Tatarla added Refactor Изменение внутреннего кода. Test Merge Candidate Игроки побудут тестировщиками. labels Dec 3, 2024
@Tatarla Tatarla requested review from homexp13 and Helg2 December 3, 2024 17:00
@homexp13
Copy link
Collaborator

homexp13 commented Dec 3, 2024

Вайдскрин увеличенный убрать надо, слишком большое преимущество по сторонам

@Dzhimax
Copy link

Dzhimax commented Dec 3, 2024

Вайдскрин увеличенный убрать надо, слишком большое преимущество по сторонам

так оно же у всех, пофиг, не? Больше раздражает то, что сверху и снизу мешает смотреть панель с абилками и панель со шмотками, руками и интентами

@definitelynotspaghetti
Copy link
Contributor Author

Вайдскрин увеличенный убрать надо, слишком большое преимущество по сторонам

не добавлял его

@homexp13
Copy link
Collaborator

homexp13 commented Dec 3, 2024

Вайдскрин увеличенный убрать надо, слишком большое преимущество по сторонам

не добавлял его

видать память на цифры подводит

Copy link
Collaborator

@Helg2 Helg2 left a comment

Choose a reason for hiding this comment

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

Мне не нравиться этот пр, так как он оптимизирует не за счёт улучшения кода, а за счёт редактирования значений уже пост фактум.

@Helg2 Helg2 added the Do Not Merge Пр хуйня!!! Либо это ТМ. label Dec 4, 2024
@definitelynotspaghetti definitelynotspaghetti deleted the widescreen branch December 6, 2024 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge Пр хуйня!!! Либо это ТМ. Refactor Изменение внутреннего кода. Test Merge Candidate Игроки побудут тестировщиками.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants