msupply-foundation / msupply-cold-chain

Android application for viewing and monitoring temperatures of fridges
GNU General Public License v3.0
3 stars 3 forks source link

Keyboard issue with 0.5.0-rc1 #217

Closed chetstone closed 11 months ago

chetstone commented 2 years ago

Describe the bug When I try to change the name of a sensor in landscape mode, and when I insert the cursor in the text box, the keyboard does not stay up. The problem does not exist in portrait mode.

To Reproduce Steps to reproduce the behavior:

  1. With the tablet in landscape mode, Go to Settings/Sensor
  2. Choose a sensor
  3. Touch Sensor name
  4. Touch somewhere in the text box
  5. The keyboard will pop up momentarily on the screen, then disappear

Expected behavior The keyboard should stay up allowing you to edit the name

Smartphone (please complete the following information):

Additional context Problem does not occur in Portrait mode

Also, when I download the APK in chrome, it asks me if I want to replace the app, saying "this will not change any of your data". However, it did delete my configuration. I had to reconnect to my sensors and the logs were gone.

(This is not a problem for me, since I'm not using the app for anything important)

I'm not sure if this is expected or not, I don't think I've ever tried to install an APK this way on android.

mark-prins commented 2 years ago

Thanks @chetstone! Good to hear from you 😄

chetstone commented 2 years ago

How're things down under Mark?

mark-prins commented 2 years ago

Wintery for us! good weather for sitting inside and looking at cold chain issues 😉 And with you?

chetstone commented 2 years ago
        We’re having a very pleasant summer here in the mountains with lots of needed afternon thundershowers…---- On Mon, 11 Jul 2022 18:57:20 -0600  Mark ***@***.***> wrote ----  

Wintery for us! good weather for sitting inside and looking at cold chain issues 😉
And with you?

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

mark-prins commented 2 years ago

Note: fine on the older Lenovo I have

adamdewey commented 11 months ago

Just tidying up the backlog - not sure we want to support landscape as a use case so closing for now