Ability to copy admin logs

[InsaneCake]

Описание PR

Задача на борде

• Набор компонентов AdminLogsLabel заменён на единственный TextEdit
• Полученные логи хранятся в поле класса
• При изменении фильтров только пересобирается содержимое TextEdit
• Методы фильтрации теперь принимают SharedAdminLog вместо AdminLogLabel
• В логе явно указывается тип события
• Автоскролл поля вниз на самые свежие логи
• Очистка сообщений сервера от BB разметки при формировании лога
  • Очистка с помощью регулярки, в которой учитывается список используемых тегов, демо тут: https://regex101.com/r/3krK2I/2
  • Прекомпиляцию регулярного выражения пробовал, но с ней клиент не собирается.

Возвращение цветной разметки - отдельная задача, над которой продолжаю работать

Медиа

До:

После:

Проверки

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

Изменения

🆑

• tweak: Добавлена возможность выделять и копировать админ-логи
• tweak: В админ-логах явно указывается тип события

[KashRas]

По факту, из красивой менюшки сделал ты обыкновенный набор текста. Неудобства, ничего.

[InsaneCake]

По факту, из красивой менюшки сделал ты обыкновенный набор текста. Неудобства, ничего.

Так а визуально что там визуально изминилось? Разделитель-линия изменился на разделитель-пустую строку.

Это сугубо утилитарное окно, функциональность в нём должна быть первичной.
ПР решает конкретную задачу - возможность предоставлять логи не скриншотами, а простым выделением и копированием текста - его проще предоставить где угодно, хоть в чате в самой игре.
Дополнительно ещё и тип события появился, раньше его можно было определить только прочитав сообщение.

Читаемость логов, как минимум, не ухудшилась, запрашиваемый функционал появился, внешний вид изменился незначительно.

В целом, сделано как мы и условились сделать на созвоне, только оказалось, что выделение и копирование текста Robust Toolbox позволяет делать только в полях для редактирования, поэтому поле осталось без цветного форматирования.

Обсуждался вариант с кастомными компонентами, но пока так сильно не заморачиваться.

Единственная проблема этого ПР в отсутствии прокрутки лога в конец.
Подозреваю, что дело в том, что текстовое поле не растягивает скролл-контейнер, а использует прокрутку внутри себя, а программно управлять прокруткой можно только в скролл-контейнере, пока не придумал как обойти.

[Legendaxe]

Выглядит прилично, я так понимаю тэги Color и т.д. все еще на скринах описания ПР-а из-за того что ты просто забыл обновить их.

Визуальный стиль отображения логов действительно несколько изменился, но думаю администрация привыкнет.

[InsaneCake]

Выглядит прилично, я так понимаю тэги Color и т.д. все еще на скринах описания ПР-а из-за того что ты просто забыл обновить их.

Да, так и есть как дома буду, могу обновить для пущей точности.

Визуальный стиль отображения логов действительно несколько изменился, но думаю администрация привыкнет.

Там есть ещё задача на улучшение цветового форматирования (оно хоть и было, но весьма скудное), но, с текущими элементами интерфейса в движке, копирование и улучшение цветовой разметки - это взаимоисключающие задачи.
Я сейчас прорабатываю варианты решения, чтобы можно было хоть как-то получить и то и то, как будут предложения по существу - нужно будет обсудить с админами.

UPD
Обновил криншот

[Legendaxe]

Аппрувую второй раз, с крайнего аппрува из изменений только коммит на мердж апстрима, апстрим в последний раз трогал указанные файлы месяц назад так что никаких конфликтов я не наблюдаю да и не должны они появиться.