wormhole-william-mobile icon indicating copy to clipboard operation
wormhole-william-mobile copied to clipboard

Unable to type in receive codes

Open duxovni opened this issue 2 years ago • 7 comments

All of the text input fields in the app seem to break when I try to input a string matching [0-9]+-[a-z]+. For instance, if i start typing 1-a, the a never appears in the text box, all I can type is 1-. This is obviously especially problematic for the recv code field, but the send text field has the same issue.

I'm using version 1.0.8 on a Pixel 3a running LineageOS 18.1.

duxovni avatar Mar 30 '22 21:03 duxovni

(The app isn't completely unusable, I can still paste in recv codes from the clipboard, but I need to type them into a text field on a different app to copy them.)

duxovni avatar Mar 30 '22 21:03 duxovni

We made a change in 1.0.8 to automatically convert spaces to dashes (-) to make it easier to input codes. I wonder if that is causing an issue for you here. Could you try testing with 1.0.6 to see if that version also has the issue? https://github.com/psanford/wormhole-william-mobile/releases/tag/v1.0.6

psanford avatar Mar 30 '22 21:03 psanford

Just checked, 1.0.6 works fine. I'm puzzled why fields other than the recv code field would also be affected if the space conversion were the cause of the bug, but it's definitely due to some change between 1.0.6 and 1.0.8 :/

duxovni avatar Mar 30 '22 21:03 duxovni

1.0.7 also works for me.

duxovni avatar Mar 30 '22 21:03 duxovni

That makes it very likely this change is somehow responsible: https://github.com/psanford/wormhole-william-mobile/commit/07a623f353b1ed4c4c6ff593ec863eb03a3fc466

Not sure why this would behave differently on lineageOS vs stock Android 12. I'll see if if can dig in a bit more when I have some time. Worst case, we can revert that change.

psanford avatar Mar 30 '22 22:03 psanford

LineageOS 18.1 is based off Android 11 fwiw. Given that text fields other than the recv code field have the same bug, I'm not sure reverting the change to that one field will actually make a difference, but maybe if I have time at some point I'll try and set up a build environment and do some testing of my own. I appreciate you looking into this!

duxovni avatar Mar 30 '22 22:03 duxovni