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

Ошибка обработки ввода с мобильных устройств в моде protector_lott #1959

Open
blanka-kato opened this issue Jan 21, 2025 · 11 comments
Labels
confirmed low priority Пониженный приоритет задачи или PR

Comments

@blanka-kato
Copy link

blanka-kato commented Jan 21, 2025

Описание ошибки:
Блоки группы protector_lott криво обрабатывают нажатие клавиши Enter с виртуальных клавиатур смартфонов.
Ошибка не даёт возможности добавить нового участника в блок протекции protector_lott:protect2 или добавить имя защищённому сундуку protector_lott:chest

Как воспроизвести:

Шаги воспроизведения на полигоне:

Блок протекции

  • Подключиться к серверу с мобильного клиента
  • Установить блок protector_lott:protect2
  • Ударить по блоку, чтоб появилась форма ввода
  • В форму записать имя нового участника
  • Закрыть клавиатуру
  • Закрыть форму
  • Имя нового участника не сохранилось

Защищённый сундук

  • Подключиться к серверу с мобильного клиента
  • Установить сундук protector_lott:chest
  • Ударить по сундуку, чтоб появилась форма ввода
  • В форму записать имя сундука
  • Закрыть клавиатуру
  • Закрыть форму
  • Имя сундука не сохранилось

Ожидаемое поведение:

Блоки группы protector_lott должны обрабатывать нажатие клавиши Enter с виртуальных клавиатур смартфонов или потерю фокуса в текстовом поле формы, как подтверждение ввода от пользователя.

@blanka-kato blanka-kato added bug Для задач: ошибка, которую необходимо исправить confirmed labels Jan 21, 2025
@blanka-kato blanka-kato changed the title Ошибка добавления игрока к защищённой области Ошибка обработки ввода с мобильных устройств в моде protector_lott Jan 21, 2025
@alek13
Copy link
Member

alek13 commented Jan 21, 2025

  • В форму записать имя нового участника
  • Закрыть клавиатуру

и при этом не нажать Enter ?


должны обрабатывать

  • нажатие клавиши Enter с виртуальных клавиатур смартфонов.

так и работает. не? или там на смартфонах как-то по другому?

  • или потерю фокуса в текстовом поле формы

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

@alek13 alek13 added the low priority Пониженный приоритет задачи или PR label Jan 21, 2025
@blanka-kato
Copy link
Author

blanka-kato commented Jan 22, 2025

Дополнение к описанию ошибки:
Блоки группы protector_lott для подтверждения ввода в текстовое поле требуют нажатия клавиши Enter
При использовании физической клавиатуры, проблем нет: вводим текст, нажимаем Enter - готово.

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

Ошибка не даёт возможности добавить нового участника в блок протекции protector_lott:protect2 или добавить имя защищённому сундуку protector_lott:chest

vokoscreenNG-2025-01-22_11-30-19-00.00.02.875-00.00.38.041.mp4

Как воспроизвести:

Шаги воспроизведения на полигоне:

Блок протекции

  1. Подключиться к серверу с мобильного клиента
  2. Установить блок protector_lott:protect2
  3. Ударить по блоку, чтоб появилась форма ввода
  4. В форму записать имя нового участника
  5. Нажать на клавиатуре Enter
  6. Введённый текст отобразиться в форме, но данные в блок не запишутся
  7. Закрыть форму
  8. Ударить по блоку ещё раз
  9. Убедиться, что в форма не показывает данные нового участника

Защищённый сундук

  1. Подключиться к серверу с мобильного клиента
  2. Установить сундук protector_lott:chest
  3. Открыть сундук, чтоб появился его инвентарь с формой ввода имени
  4. В форму записать имя сундука
  5. Нажать на клавиатуре Enter
  6. Закрыть сундук
  7. Ударить по блоку ещё раз
  8. Убедиться, что в форма не показывает новое имя сундука

@blanka-kato
Copy link
Author

@alek13
Прошу сменить приоритет на стандартный.
Причину указал в личном сообщении.

@alek13 alek13 removed the low priority Пониженный приоритет задачи или PR label Jan 22, 2025
@alek13
Copy link
Member

alek13 commented Jan 22, 2025

спасибо огромное за видео!, осталось понять как его воспроизводить локально чтобы пофиксить )))))

@alek13
Copy link
Member

alek13 commented Jan 22, 2025

Solution (workaround):

  • add additional button add for additional ability to add player without pushing Enter on mobile keyboard

@blanka-kato
Copy link
Author

спасибо огромное за видео!, осталось понять как его воспроизводить локально чтобы пофиксить )))))

Подгрузил локально

@alek13 alek13 removed the bug Для задач: ошибка, которую необходимо исправить label Jan 27, 2025
alek13 added a commit to alek13/lord that referenced this issue Jan 27, 2025
@alek13
Copy link
Member

alek13 commented Jan 27, 2025

сама обработка кнопки Enter на мобильных скорее всего закопана далеко в движке и вряд ли движок предоставляет хоть что-то по этому поводу (догадка, не проверял).
В любом случае на формочке появилась кнопка для добавления ( #1965 ),
по сему приоритет снижаем до маленького.

@alek13 alek13 added the low priority Пониженный приоритет задачи или PR label Jan 27, 2025
@Doloment
Copy link
Contributor

@alek13 такое есть

field_enter_after_edit[<name>;<enter_after_edit>](https://api.luanti.org/formspec/#field_enter_after_editnameenter_after_edit)
	• Experimental, may be subject to change or removal at any time.
	• Only affects Android clients.
	• <name> is the name of the field.
	• If <enter_after_edit> is true, pressing the "Done" button in the Android text input dialog will simulate an Enter keypress. Defaults to false when not specified (i.e. no tag for a field).

@alek13
Copy link
Member

alek13 commented Jan 27, 2025

@alek13 такое есть

нужно попробовать, только мне проверить не на чем.

@blanka-kato , я "вслепую" добавлю это решение, если на тестовом всё норм будет, закроете руками баг тогда.

@alek13
Copy link
Member

alek13 commented Jan 27, 2025

cba2457

@blanka-kato
Copy link
Author

Пометку у себя сделайте, что решение экспериментальное и может отвалиться по щелчку разрабов Luanti.
Судя по описанию выше.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
confirmed low priority Пониженный приоритет задачи или PR
Projects
None yet
Development

No branches or pull requests

3 participants