AquaMail Forum

English - Android => Development builds => Topic started by: mikeone on August 21, 2014, 08:06:25 pm

Title: Fixed! - Issue with 1.4.1-58-beta8.14
Post by: mikeone on August 21, 2014, 08:06:25 pm
Sorry but I got an issue with the latest beta version 8.14 if "swiping in message view" is activated.
First swipe is going well but second swipe stops the app.
-> screenshot

This will not happen with beta 8.13 (!)

@Kostya
Since there are no other changes maybe something went wrong with the implemention of
"recreate message web view during view recycling to avoid carrying over zoom factor"?

Title: Re: Issue with 1.4.1-58-beta8.14
Post by: Kostya Vasilyev on August 21, 2014, 08:54:55 pm
Specific to 1) message swiping being enabled 2) Android 4.0 - 4.3 inclusive,  but not 2.* or 4.4.

Already got the crash log, going to look at it now.

>> recreate message web view during view recycling to avoid carrying over zoom factor

Yes, this is the cause. Android 4.4 and 4.0-4.3 need a different song and dance here.
Title: Re: Issue with 1.4.1-58-beta8.14
Post by: Kostya Vasilyev on August 21, 2014, 09:36:49 pm
Fixed (to the extend of my testing) in -beta8.16

http://www.aqua-mail.com/download/AquaMail-market-1.4.1-58-beta8.16.apk
Title: Re: Issue with 1.4.1-58-beta8.14
Post by: mikeone on August 21, 2014, 09:53:16 pm
Fix confirmed
 :)
Thanks a lot.
Title: Re: Fixed! - Issue in 1.4.1-58-beta8.14
Post by: Kostya Vasilyev on August 22, 2014, 12:10:29 am
Great :)
Title: Reopened! - Issue with 1.4.1-58-beta8.14
Post by: mikeone on August 27, 2014, 10:01:00 am
I just got another issue with message swiping on Android 4.3.:

Enabled option "Show linked images only from known sender" in Settings / Message view / Spam control and  enabled "Swiping in message view".

In a message received from an unknown sender you will get the option "Show linked images" on the top of the message text. Tapping on this notice will show embedded images.

Now swiping to the next / previous message and came back to the original message you stil see the embedded images (and the option "Save as known sender" with the tool icon on the right edge). All works fine...right now.

But if I am swiping to the next / previous message *twice* (...or even more) and then came back to the original message the notice with the tool icon is still there but tapping on it doesn't work...it is blocked.
-> turning to message list and return to message view will "unblock" the notice/tool icon.

I went back to previous builds and find out that this behaviour was not evident in build 1.4.1-58-beta8.13 but with 1.4.1-58-beta 8.14 and newer.

Maybe this is also an issue with implementation of "recreate message web view during view recycling to avoid carrying over zoom factor" in beta8.14...?
Title: Re: Reopened! - Issue with 1.4.1-58-beta8.14
Post by: Kostya Vasilyev on August 27, 2014, 02:52:27 pm
Yes, I'm sure it is related to recycling message views. I'm going to take a look.
Title: Re: Reopened! - Issue with 1.4.1-58-beta8.14
Post by: Kostya Vasilyev on August 27, 2014, 08:00:31 pm
Fixed. BTW - it wasn't new... has been there ever since I implemented message swiping.