daid / EmptyEpsilon

Open source bridge simulator. Build with the SeriousProton engine.
https://daid.github.io/EmptyEpsilon/
GNU General Public License v2.0
528 stars 177 forks source link

No keyboard appearing in mobile #392

Closed lofrequency closed 4 years ago

lofrequency commented 8 years ago

Cannot type any inputs into the mobile version anywhere on the client or server creation screens. (dunno if this is the right place to even report this)

Using 6.0.1 on Nexus 5. My friend has the same problem; she's running 6.0.1 on a Note 7.

aaroneiche commented 7 years ago

I can confirm that I'm running into the same problem on a Galaxy S7 - no keyboard appears when focusing on the LAN field in the client.

kwadroke commented 7 years ago

I believe that's a problem with SFML's Android support.

aaroneiche commented 7 years ago

Is there a workaround? I haven't been able to figure out any solutions unfortunately. :P

daid commented 7 years ago

Actually, SFML allows you to show the virtual android keyboard, the code is just not calling it. http://www.sfml-dev.org/documentation/2.4.2/classsf_1_1Keyboard.php#ad61fee7e793242d444a8c5acd662fe5b

PatrickGrace commented 4 years ago

Was just testing the Android version and the keyboard is still disabled, thus preventing users from typing in the server IP address. Bug demonstrated in the video linked below.

I assume since this is a major game-breaking bug and it's been open for 4 years that the game is no longer under active development. Too bad as we were looking forward to a teambuilding event, and now it's gonna be on Artemis.

https://drive.google.com/file/d/1AWxcK5LIzSqkmBYNoOeaDWvRaaBoNhuu/view?usp=sharing

aBlueShadow commented 4 years ago

There is still developement going on, (just look at the date of the latest commits). But the android version is hard to test (especially as the main developer has not many devices to test it on). Also keep in mind that it is only a game breaking bug under specific circumstances. Edit: I did a quick search and have not found a recent android version of Artemis, so it does not look like the situation is much better there (maybe the reasons for that are even similar to the ones here - hard to test, small developer basis.)

PatrickGrace commented 4 years ago

Apologies - hadn't looked at commits, and realize this is not technically game-breaking. But I've been pro QA for 5 years and it's pretty much the worst user experience possible without being game-breaking. Because I happen to have the resources here right now, I tested this on about 5 different Android devices, ranging in age and size, and the exact same bug comes up for every one. I'm trying to get our devs interested in the game so they'll look into the code to fix it, but can't really hook em if I can't connect the devices to play, and we don't have enough Windows laptops in our inventory to set up on PCs only. How would you suggest I bypass the bug?

I wouldn't mind donating some hours to QA the game, but I can barely justify using our company equipment right now to test this for a company teambuilding event. I couldn't justifiably use my company equipment to offer QA services to 3rd parties. But I do have some expertise and a few Androids of my own I could offer if you need a bit of help on the QA side of things... ;-)

And you're right about Artemis - it seems to be in dev hell right now, which is why I had such high hopes for EE - both up to date AND open source -- fantastic. But for my purposes this is bug is a showstopper. Quintet is seemingly in a similar boat (starship?) as well. :-(

Anyway, my apologies if my last comments came off as flippant. Honestly I didn't think there'd be a response. But I'm happy to help where I can if you could use it.

daid commented 4 years ago

If you are depended on the android version, then you are going to have a bad time. This minor usability inconvenience isn't the worst problem on android. I see the android version is no longer marked as [BETA] on the website, it used to say that, so I must have messed something up there.

And as a so called "QA with 5 years of experience", you should know that entering an IP address is horrible UX anyhow. Which is why it does LAN discovery and has the ability for servers to register with a master server for discovery.

oznogon commented 4 years ago

anyway I'm setting up an Android build environment for the first time, and if it works I'll test the sf::setVirtualKeyboardVisible() calls

I think nobody worked on this for 4 years because nobody reported it for 4 years (and for one of those years there was a breaking bug in SFML on that function)

oznogon commented 4 years ago

I've got a build environment, and I've got the virtual keyboard appearing for the server password prompt. I don't have a great understanding of the UI so I haven't figured out yet how to detect when the IP field is focused in order to trigger the keyboard there; the password prompt is tied to the game client state, which is easier.

oznogon commented 4 years ago

Just went at it from a lower level and toggled it when any text entry has focus, which makes sense in theory.

kwadroke commented 4 years ago

Put a Toggle Keyboard button on those screens, maybe? It's possible that there might be a keyboard paired with the Android device and the on screen one may or may not be needed.

oznogon commented 4 years ago

@PatrickGrace As a worst-case alternative if a fix doesn't ship in time for your team-building exercise, the Hacker's Keyboard app (which I use, and which is also available on F-Droid), has an option in its Settings ("Use permanent notification" under "Input mode settings") that allows the keyboard to be invoked from an optional permanent notification.

PatrickGrace commented 4 years ago

Agreed IP entry is not the best experience, but I've often found more complicated solutions to be even more frustrating when for some reason they don't automagically work. Sometimes a good old fashioned dumb button is the truely "smart" solution. Especially when trying to deal with an overly complicated corporate network structure. I just have no way of forcing the keyboard to come up, and the only workaround I had found was attaching a bluetooth keyboard, but bluetooth is not really stable when surrounded by 250 other users all pumping out their own wifi and bluetooth signals.

On Wed, Feb 12, 2020 at 5:32 PM daid notifications@github.com wrote:

If you are depended on the android version, then you are going to have a bad time. This minor usability inconvenience isn't the worst problem on android. I see the android version is no longer marked as [BETA] on the website, it used to say that, so I must have messed something up there.

And as a so called "QA with 5 years of experience", you should know that entering an IP address is horrible UX anyhow. Which is why it does LAN discovery and has the ability for servers to register with a master server for discovery.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/daid/EmptyEpsilon/issues/392?email_source=notifications&email_token=AKWTGXEJRPISNSNY25K7ZS3RCR2JPA5CNFSM4COQ6B4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELSUTTY#issuecomment-585451983, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKWTGXE3AWUJCGUXAVRFRH3RCR2JPANCNFSM4COQ6B4A .

-- Patrick Grace QA Analyst www.ProdigyGame.com http://www.prodigygame.com/ 1100 Burloak Drive Suite 200 Burlington ON L7L 6B2

PatrickGrace commented 4 years ago

The bluetooth solution does work, it's just not as easily done. I was hoping to have people come to the meeting room and find all the consoles set up already. I could do this with Bluetooth, but pairing and unpairing 10 or 15 devices does not sound fun. It is a viable workaround though, and I thank you for the suggestion. :-)

On Thu, Feb 13, 2020 at 12:05 AM oznogon notifications@github.com wrote:

@PatrickGrace https://github.com/PatrickGrace As a worst-case alternative if a fix doesn't ship in time for your team-building exercise, the Hacker's Keyboard app https://play.google.com/store/apps/details?id=org.pocketworkstation.pckeyboard&hl=en_US (which I use, and which is also available on F-Droid https://f-droid.org/en/packages/org.pocketworkstation.pckeyboard/, has an option in its Settings ("Use permanent notification" under "Input mode settings") that allows the keyboard to be invoked from an optional permanent notification.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/daid/EmptyEpsilon/issues/392?email_source=notifications&email_token=AKWTGXG42IJZV2VGNOHE2NDRCTIK7A5CNFSM4COQ6B4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELTNGBY#issuecomment-585552647, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKWTGXGM3J5NBXLDYR75SLTRCTIK7ANCNFSM4COQ6B4A .

-- Patrick Grace QA Analyst www.ProdigyGame.com http://www.prodigygame.com/ 1100 Burloak Drive Suite 200 Burlington ON L7L 6B2

PatrickGrace commented 4 years ago

I used to use Hacker Keyboard as my primary keyboard for years on an older Android. This is a great idea! Thank you!

On Thu, Feb 13, 2020 at 12:05 AM oznogon notifications@github.com wrote:

@PatrickGrace https://github.com/PatrickGrace As a worst-case alternative if a fix doesn't ship in time for your team-building exercise, the Hacker's Keyboard app https://play.google.com/store/apps/details?id=org.pocketworkstation.pckeyboard&hl=en_US (which I use, and which is also available on F-Droid https://f-droid.org/en/packages/org.pocketworkstation.pckeyboard/, has an option in its Settings ("Use permanent notification" under "Input mode settings") that allows the keyboard to be invoked from an optional permanent notification.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/daid/EmptyEpsilon/issues/392?email_source=notifications&email_token=AKWTGXG42IJZV2VGNOHE2NDRCTIK7A5CNFSM4COQ6B4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELTNGBY#issuecomment-585552647, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKWTGXGM3J5NBXLDYR75SLTRCTIK7ANCNFSM4COQ6B4A .

-- Patrick Grace QA Analyst www.ProdigyGame.com http://www.prodigygame.com/ 1100 Burloak Drive Suite 200 Burlington ON L7L 6B2

daid commented 4 years ago

Or use the release of yesterday which contains the fix...

PatrickGrace commented 4 years ago

LOL an ever better option! Thank you!

On Wed, Feb 19, 2020 at 5:05 PM daid notifications@github.com wrote:

Or use the release of yesterday which contains the fix...

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/daid/EmptyEpsilon/issues/392?email_source=notifications&email_token=AKWTGXDOE6U2XF2KUBZSQDLRDWULNA5CNFSM4COQ6B4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMJ3YEA#issuecomment-588495888, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKWTGXC7MDL6NE2IGS4T443RDWULNANCNFSM4COQ6B4A .

-- Patrick Grace QA Analyst www.ProdigyGame.com http://www.prodigygame.com/ 1100 Burloak Drive Suite 200 Burlington ON L7L 6B2