-
Notifications
You must be signed in to change notification settings - Fork 128
Великая переработка панели эмоций! #520
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
base: master
Are you sure you want to change the base?
Conversation
|
А что за ошибка.. |
tiktok__1750025158993_1ee36ef9-9b9d-4243-b7e0-f55874eb279c_mp4_mp4_-_1.mp4 |
|
Киси милые :3 |
|
@AndrewFenriz |
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
# Conflicts: # Resources/Prototypes/Entities/Mobs/Cyborgs/base_borg_chassis.yml # Resources/Prototypes/Entities/Mobs/Species/base.yml # Resources/Prototypes/_DeadSpace/Entities/Mobs/Species/xenomorph.yml
KloopRE
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Я не особо вникал в код, но если работает, то хорошо. Ты же вроде не умеешь его писать? Так как тогда?
|
|
||
| public override void Initialize() | ||
| { | ||
| base.Initialize(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Зачем?
| AppendStyleClass="{x:Static style:StyleBase.ButtonSquare}" | ||
| /> | ||
| <!-- <ui:MenuButton | ||
| <ui:MenuButton |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
edit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Не требуется, т.к. это раскомментирована строка оффов
По большей части велосипед изобретать не нужно было. Я взял за основу визардскую панель, которая по всем фронтам лучше и перенёс несколько положительных моментов с нашей устаревшей. Что именно - я уже не вспомню, PR висит очень долго |
|
Сейчас думаю, что после апстрима будет проще просто взять официальную и по мелочи её доработать |
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
# Conflicts: # Resources/Locale/ru-RU/_deadspace/chat/emotes.ftl # Resources/Prototypes/_DeadSpace/Voice/speech_emotes.yml
|
По итогу я полностью снёс кривую панель АДТ, улучшил оффовскую в меру сил и возможностей |



Описание PR
Панель эмоций была переработана - за основу взят интерфейс новой, стильной и аккуратной панели. Сохранена фильтрация и лучшие аспекты предыдущей от ADT. Убрана ненавистная многими кнопка действия открытия эмоций - теперь это адекватно работает через хоткей или кнопку сверху. Были добавлены недостающие картиночки для эмоций фелинидов, вульп, слаймолюдов и чихания. Исправлены некоторые эмоции, которые не были локализированы. Расхардкожено скрытие эмоций - теперь это можно сделать одной строкой в прототипе эмоции. В скрытые добавлена эмоция "задыхается". Переработаны иконки эмоций слаймов - теперь они стильные и красивые. Мелкие правки эмоций, подробнее в изменениях. Тэг кпб добавлен некоторым видам эмоций, для перестраховки
Делал на основе ADT, но можно было бы и оригинальные файлы изменить - тогда все упоминания ADT панели можно удалять
Почему / Зачем / Баланс
Официальные разработчики давно завезли красивую панельку, вместо просто её включения решил сделать гибрид. Во благо чувства прекрасного, для удобства и избавления от кнопки действия, которую я скрываю каждый раунд на протяжении полутора лет.
Технические детали
Были изменены серверная и клиентская системы панели от ADT, удалён компонент панели, сам файл переименован в ивенты. Раскомментирована кнопка панели эмоций.
Медиа
Требования
Критические изменения
Нет?
Список изменений
🆑