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

[Admin] Синхронизация цвета имени администратора с настройками OOC чата #929

Merged

Conversation

Schrodinger71
Copy link
Collaborator

@Schrodinger71 Schrodinger71 commented Jan 1, 2025

Описание PR

Теперь в админ-ахелпе и OOC чате используется одинаковый цвет для имени администратора, который был установлен через команду setadminooc. Цвет имени администратора в чате теперь синхронизирован с цветом, заданным в настройках OOC.

Почему / Баланс

Это изменение улучшает визуальное восприятие администраторов в игре. Ранее, имя администратора в админ-ахелпе не использовало его индивидуальный цвет, установленный в setadminooc.

Техническая информация

Внесены изменения в код для синхронизации цвета имени администратора в админ-ахелпе с настройкой цвета, установленной через команду setadminooc. Цвет теперь задается с учетом предпочтений администратора, и если цвет не установлен, применяется стандартный красный.

Медиа

image
image

Требования

  • Я прочитал(а) и следую Руководство по созданию пулл реквестов. Я понимаю, что в противном случае мой ПР может быть закрыт по усмотрению мейнтейнера.
  • Я добавил скриншоты/видео к этому пулл реквесту, демонстрирующие его изменения в игре, или этот пулл реквест не требует демонстрации в игре

Критические изменения

  • Нет критических изменений.
    Content.Server\Administration\Systems\BwoinkSystem.cs
    затронут только этот файл, все изменения подписаны.

Чейнджлог
🆑 Шрёдька

  • add: Теперь используется цвет, установленный для администратора через команду setadminooc, для его ника в админ-ахелпе.

@Schrodinger71 Schrodinger71 merged commit 06a7f81 into AdventureTimeSS14:master Jan 1, 2025
9 of 10 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.

1 participant