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

[Promises, Part 1] TGUI Admin tickets #13631

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

AzzyDreemurr13
Copy link
Contributor

@AzzyDreemurr13 AzzyDreemurr13 commented Nov 13, 2024

Описание изменений

Как-то раз пока был Админом, я в ЦК заикнулся про ТГУИ тикеты. В общем, пока ещё не успел уйти в одно место, начинаю выпускать партию ПРов в которых я обещал что-то сделать. Один из них ТГУИ панелька ahelp!
IMG_20241113_182250_626

Почему и что этот ПР улучшит

Переносим интерфейсы на ТГУИ формат по плану переносов всех UI на ТГУИ

Авторство

Моё (портировано с VOREStation/VOREStation#13845)

Чеинжлог

🆑

  • rscadd: Добавлен TGUI интерфейс для Админ Тикетов

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича UI User Interface labels Nov 13, 2024
@AzzyDreemurr13 AzzyDreemurr13 added the Admin Всё что касается админов. Логирование, инструментарий и т.д. label Nov 13, 2024
code/modules/admin/verbs/adminhelp.dm Outdated Show resolved Hide resolved
Comment on lines +564 to +565
if("legacy")
TicketPanelLegacy()
Copy link
Contributor

Choose a reason for hiding this comment

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

если и оставлять легаси-уи то мб в префах это сохранять

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ну тут на вкус и цвет, не думаю что критично. Подожду мнения других мейнтеров пока что

</LabeledList.Item>
)}
<LabeledList.Item label="Actions">
<div dangerouslySetInnerHTML={{ __html: actions }} />
Copy link
Contributor

Choose a reason for hiding this comment

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

а в чем смысл тгуи-обертки если мы по-прежнему пользуемся сомнительным dangerouslySetInnerHTML?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

У тебя есть предложения как это можно реализовать лучше?

Copy link
Contributor

Choose a reason for hiding this comment

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

реализовать то как и все остальные тгуи интерфейсы
без dangerouslySetInnerHTML
у тебя буквально в этом же интерфейсе используются тгуишные кнопки

@AzzyDreemurr13
Copy link
Contributor Author

И небольшой факт, так как ТГУИ, то теперь не надо будет прожимать по КД рефрешь, все моментально появится в тикете

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin Всё что касается админов. Логирование, инструментарий и т.д. Feature Новая фича UI User Interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants