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

Autoform refactor #2492

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

Conversation

Kirus59
Copy link
Collaborator

@Kirus59 Kirus59 commented Jan 16, 2025

Описание PR

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

Медиа

2025-01-16.21-11-59.mp4

Если на Id-карте иное имя - то его выведет отдельно
image

Проверки

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

Изменения

🆑

  • add: Добавлен UI для помощи в заполнении документов
  • remove: Убраны теги %job и %name

@github-actions github-actions bot added Changes: UI Изменение затронуло файлы ".xaml" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: C# Изменение затронуло файлы ".cs" labels Jan 16, 2025
@stalengd stalengd self-assigned this Jan 16, 2025
@Kirus59
Copy link
Collaborator Author

Kirus59 commented Jan 17, 2025

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

@Kirus59 Kirus59 marked this pull request as ready for review January 17, 2025 18:42
@Kirus59 Kirus59 requested a review from stalengd January 17, 2025 18:42
@github-actions github-actions bot added Status: Needs Review Merge Conflict Some content has conflicts. labels Jan 17, 2025
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Merge Conflict Some content has conflicts. label Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: UI Изменение затронуло файлы ".xaml" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants