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

Bug: Mod sort text field hard to use on Android in some orientations. #12331

Open
1 task done
The-Talik opened this issue Oct 21, 2024 · 2 comments
Open
1 task done
Labels

Comments

@The-Talik
Copy link

The-Talik commented Oct 21, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Game Version

4.13.18

Describe the bug

So, this one took me a while to realize what was happening, but if my phone is what I consider upside-down-landscape, the mod filter text field operates in an intuitive way -- I get a keyboard at the bottom of my screen, and the text gets added to the input field.
If my phone is right-side-up-landscape, then when I click the filter text, I get a popup "Enter search text". I can enter the text, but clicking OK doesn't update the text in the filter input, making it impossible to use.
image
image

Expected behavior would be identical behavior no matter the phone orientation.

Steps to Reproduce

On mobile, go to mod page. Click filter. Turn phone upside down, click filter again.

Screenshots

No response

Link to save file

No response

Operating System

Android

Additional Information

No response

@The-Talik The-Talik added the bug label Oct 21, 2024
@yairm210
Copy link
Owner

@SomeTroglodyte This looks like your expertise

@SomeTroglodyte
Copy link
Collaborator

Not really. That textbox was originally by - what was the name - Timo. I only refactored code structure afterwards. The idea "scroll the field into view, because popping up the soft keyboard might have hidden it, but if you can't, then show an extra popup instead - that was Azzurite aka Timo T. Oh - I even mentioned that I tried to provoke this box when testing #11705 but couldn't trigger it... I mean, "turn phone upside down", that requires a little creativity.

Here, there's two "failures": recognizing that the field is already in view, and transmitting content back to the underlying field on "OK". Sounds debuggable if "upside down" is a reliable trigger - but I haven't got the energy atm. Back then I briefly considered simply disabling that popup - comment out private inner class TextfieldPopup and line 88 where it is instantiated - because I couldn't get it to kick in, but decided against as I don't know about parts of the multiplayer UI potentially depending on it.

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

4 participants
@The-Talik @yairm210 @SomeTroglodyte and others