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

TgUI (TgUI input) не принимает достаточно длинные строки. #1292

Closed
msw7007 opened this issue Jun 6, 2024 · 4 comments
Labels
Под вопросом Необходима дополнительная информация.

Comments

@msw7007
Copy link

msw7007 commented Jun 6, 2024

BYOND Version

No response

Описание проблемы

При вводе длинных строк при использовании tgUI если их размеры превышают опредленное (скорее всего внутреннее) заданое число символов в окне ввода, кнопка submit не отрабатывает (не происходит вызов процедур, окно не закрывается).
Данная проблема замечена в медицинских записях и при попытке импорта мелодий для музыкальных инструментов.

Что должно было произойти?

Информация должна быть передана в процедуру на ее обработку.

Что случилось вместо этого?

Ничего, окно не функционирует

Почему это плохо/Какие последствия?

Нет возможности вводить длинные медицинские записи, что сказывается на РП. Нет возможности полноценного импорта мелодий в инструменты, что негативно сказывается на игровом опыте, который связан с музыкой.

Шаги для повторения проблемы.

Взять строку длиной значительно более 1024 (почему так опишу позже).
Взять в руки муз. инструмент.
Нажать импорт
Вставить строку.
Нажать Submit

Когда проблема началась?

Недели 2 назад импорт работал (не уверен, что он был через tgUI), обратил внимание на проблему примерно в районе 31-ого мая 2024

Дополнительная информация

При нажатии на submit должна вызываться соответствующая функция. Тесты через VSC на примере проблемы импорта мелодий показали что при нажатии submit и длиной строки в районе 2к символов (привожу строку) не происходит вызова функции /datum/tgui_input_text/ui_act(action, list/params) из text_input.dm

При этом при вставке например текста в 500 символов вызов происходит.
Более точные тесту указали на длину в 1283 символа, если текст изначально представлен одной строкой (1283 символа отгружаются, 1284 нет) и 1243 если мультистрокой (полагаю, что это связано с тем, что используются символы переноса каретки на следующую строку).

Данная информация представлена в рамках форка билда при локальнйо установке.
На данный момент для игроков есть решение отключить tgUi Input (но он классный).

Ноты Born to be Wild P1.txt

Связанные логи

No response

@dj-34
Copy link
Collaborator

dj-34 commented Jul 11, 2024

Актуально?

@dj-34 dj-34 added the Под вопросом Необходима дополнительная информация. label Jul 11, 2024
@skifsky
Copy link

skifsky commented Aug 17, 2024

У оффов тг тоже есть такая проблема, похоже это связано с новыми версиями Бьенда tgstation/tgstation#83722 (comment)

@skifsky
Copy link

skifsky commented Aug 17, 2024

Проверил на локалке, видимо уже неакутально (по крайней мере для версии 515.1642)

@dj-34
Copy link
Collaborator

dj-34 commented Sep 4, 2024

Видимо, не актуально.

@dj-34 dj-34 closed this as completed Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Под вопросом Необходима дополнительная информация.
Projects
None yet
Development

No branches or pull requests

3 participants