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

Tweak: Smooth with airlocks #691

Closed
wants to merge 1 commit into from
Closed

Tweak: Smooth with airlocks #691

wants to merge 1 commit into from

Conversation

AyIong
Copy link
Collaborator

@AyIong AyIong commented Nov 13, 2023

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

Теперь стены и окна сглаживаются с аирлоками, выглядит красивее.
Так же отключил сглаживание окон с деревянными стенами.
Вообще, тут надо делать пересортировку групп на оффы, но мне лень и я не знаю надо ли им это, учитывая как быстро они откатили сглаживание с шлюзами, и как долго никто даже не пытается возродить это.

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

КРАСИВО

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

image

image

image

image

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

Changelog

🆑
tweak: Аирлоки теперь соединяются со стенами и окнами
/:cl:

@dj-34
Copy link
Collaborator

dj-34 commented Nov 13, 2023

НЕ НАДО ДЯДЯ
Это не красиво

@AyIong
Copy link
Collaborator Author

AyIong commented Nov 13, 2023

НЕ НАДО ДЯДЯ Это не красиво

Ставьте реакты под ПР если не нравится, голосование ёпты

@dj-34
Copy link
Collaborator

dj-34 commented Nov 13, 2023

Вот стены здорового человека, где они не продолжаются в аирлок

image

Стены курильщика, которые продолжаются в аирлок

image

@AyIong
Copy link
Collaborator Author

AyIong commented Nov 13, 2023

Вот стены здорового человека, где они не продолжаются в аирлок

image

Стены курильщика, которые продолжаются в аирлок

image

Ну так класно же, аирлок будто в стене а не просто стоит между стенами

@dj-34
Copy link
Collaborator

dj-34 commented Nov 13, 2023

Вот стены здорового человека, где они не продолжаются в аирлок
image
Стены курильщика, которые продолжаются в аирлок
image

Ну так класно же, аирлок будто в стене а не просто стоит между стенами

А опирается он на что? Без сглаживания, края стены ощущаются как "рамки" для аирлока

@AyIong
Copy link
Collaborator Author

AyIong commented Nov 13, 2023

А опирается он на что? Без сглаживания, края стены ощущаются как "рамки" для аирлока

На стену опирается, он же "в стене"
А вот о что он упирается без сглаживания, не понятно

@dj-34
Copy link
Collaborator

dj-34 commented Nov 13, 2023

А опирается он на что? Без сглаживания, края стены ощущаются как "рамки" для аирлока

На стену опирается, он же "в стене" А вот о что он упирается без сглаживания, не понятно

так я говорю же, без сглаживания, края стены превращаются в рамки для аирлока

image

"В какие-то хуёвые рамки оно превращается" (c) Aylong

@AyIong
Copy link
Collaborator Author

AyIong commented Nov 13, 2023

Аирлок не дверь, он открывается В стену а не в сторону

@dj-34
Copy link
Collaborator

dj-34 commented Nov 13, 2023

Аирлок не дверь, он открывается В стену а не в сторону

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

Copy link
Collaborator

@PhantornRU PhantornRU left a comment

Choose a reason for hiding this comment

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

Одобряю. Мне тоже не нравится что аирлоки стояли чужеродно словно между стенами.
Вдовесок если вдруг АИРЛОКи переспрайтят, они сразу будут сглажены как положено. Но смотрится уже лучше по моему мнению.

@AyIong
Copy link
Collaborator Author

AyIong commented Nov 14, 2023

Одобряю. Мне тоже не нравится что аирлоки стояли чужеродно словно между стенами. Вдовесок если вдруг АИРЛОКи переспрайтят, они сразу будут сглажены как положено. Но смотрится уже лучше по моему мнению.

Вижу вы человек высокой культуры

@AyIong
Copy link
Collaborator Author

AyIong commented Nov 17, 2023

Увы

@AyIong AyIong closed this Nov 17, 2023
@AyIong AyIong deleted the SMOOOTH branch November 17, 2023 15:10
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