Android build -780

[Dirk Hohndel]

On Sat, Feb 06, 2016 at 07:21:36PM +0200, Willem Ferguson wrote:
> I have been doing some tests with build 780 (Qt 5.5) on Android 4.3 (Samsung
> S3), APK does not run on Android 5.1 (Samsung S5).

Hmm. Lovely.

> Most of the features appear ok. Tested downloads of log as well as GPS
> fixes, delete of GPS fixes, view of GPS localities and of dives, view and
> scrolling of profiles, preferences. The problems that I found were mostly
> with hand-entering a dive. The unresponsive behaviour of the silly button as
> well as the menus in the left-hand slide-out was gone.

Good. By the way... it's called the Action Button. Just in case you feel
silly calling it the silly button :-)

> 1) When trying to check the box to show the password, the checkbox was
> mostly unresponsive and only responded after the password had been typed in.

That's a new one. It works for me every time, tested on two devices. I
need to find my Samsung Galaxy S III again - somehow that got lost and
it's my only Samsung device with their brilliant UI "improvements".

> 2) After loading the dive list, the device was accessing the Internet (do
> not know why). During this process which takes about 15 sec, the software
> was mostly unresponsive.

This is when it synchronizes with the cloud storage. There are a few more
things around performance that need improvement - I have noticed this
specific delay as well.

> When hand-entering a dive:
> 3) Editing text in a text box was a pain if one attempted to drag the edit
> cursor around. Mostly it just did not work, the cursor refused to move. On
> other occasions it moved back a character or 2 and then refused to move
> further. The only way was to delete text all the way to the edit point and
> start again. The same applied to correcting the email in the login data
> screen.

So this is definitely a Qt/QML issue since all we do is use their widgets.

> 4) If the duration of the hand-entered dive is typed as "37 min" then
> Subsurface-mobile sets it to zero. It MUST have no spaces between the number
> and the "min".

Ok, that should be easy to fix... just relax the parser a little. I'll
look into that.

> 5) I could not get a hand entered dive profile to show. See attached image.

That seems like the issue that has been described a few times before. Best
guess is a "race condition". You can see that the axes are drawn already,
but the profile hasn't been drawn, yet. The profile is rendered before it
is finished drawing. I have no clue how to fix that because of the weird
object oriented ass backwards way the profile code works.

/D