Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Regression] Crash on screen rotation when composing email #8606

Open
2 tasks done
xandro0777 opened this issue Nov 26, 2024 · 19 comments
Open
2 tasks done

[Regression] Crash on screen rotation when composing email #8606

xandro0777 opened this issue Nov 26, 2024 · 19 comments
Labels
type: bug Something is causing incorrect behavior or errors type: crash Issues that involve fixing a crash or ANR

Comments

@xandro0777
Copy link

Checklist

  • I have used the search function to see if someone else has already submitted the same bug report.
  • I will describe the problem with as much detail as possible.

App

Thunderbird for Android

App version

8.1

Where did you get the app from?

F-Droid

Android version

13

Device model

Realme

Steps to reproduce

  1. In unified mailbox, start composing an email
  2. Enter a valid To address
  3. Rotate device 90° counterclockwise
  4. Rotate device back to normal position
  5. Crash, composed email is gone without trace

Sometimes, instead of crashing the draft disappears and the App returns to unified mailbox

Expected behavior

Don't crash

Actual behavior

Crash or composed email lost without trace

Logs

No response

@xandro0777 xandro0777 added type: bug Something is causing incorrect behavior or errors unconfirmed Newly reported issues awaiting triage or confirmation labels Nov 26, 2024
@xandro0777
Copy link
Author

k9mail-logs.txt

@xandro0777 xandro0777 changed the title Crash on screen rotation when composing email [Regression] Crash on screen rotation when composing email Nov 26, 2024
@xandro0777
Copy link
Author

Did not happen on any previously installed versions

@xandro0777
Copy link
Author

No trace of the email in the drafts folder no matter how long I have been composing it.

@kewisch kewisch added type: crash Issues that involve fixing a crash or ANR and removed unconfirmed Newly reported issues awaiting triage or confirmation labels Nov 26, 2024
@kewisch
Copy link
Member

kewisch commented Nov 26, 2024

@kewisch kewisch added this to the Thunderbird 9.0 milestone Nov 26, 2024
@cketti
Copy link
Member

cketti commented Nov 26, 2024

I'm not able to reproduce this crash with the given information.

@xandro0777: Can you provide more details? What email address do you enter in step 2? Where's the cursor when you rotate the device? Can you record a video of such a crash?

@cketti cketti added the status: needs information Needs more information to proceed label Nov 26, 2024
@xandro0777
Copy link
Author

xandro0777 commented Nov 26, 2024 via email

@github-actions github-actions bot added status: answered The issue was answered and is waiting for maintainer review. and removed status: needs information Needs more information to proceed labels Nov 26, 2024
@xandro0777
Copy link
Author

xandro0777 commented Nov 26, 2024 via email

@cketti
Copy link
Member

cketti commented Nov 26, 2024

From the data we have via Google Play this crash does not seem to be limited to Android 13. However, it isn't reported nearly enough for this to happen on every device rotation when composing a message (which explains my trouble reproducing the crash).

In order to fix the crash, we need to be able to reproduce it on one of our test devices or an emulator. So someone needs to provide instructions on how to reliably reproduce this crash. Please don't make any assumptions about what information is or isn't relevant. Include as much detail as possible. A video could be useful because it shows the user interaction as it happened without omitting any seemingly unimportant details.

@cketti cketti added status: needs information Needs more information to proceed and removed status: answered The issue was answered and is waiting for maintainer review. labels Nov 26, 2024
@xandro0777
Copy link
Author

xandro0777 commented Nov 26, 2024 via email

@github-actions github-actions bot added status: answered The issue was answered and is waiting for maintainer review. and removed status: needs information Needs more information to proceed labels Nov 26, 2024
@xandro0777
Copy link
Author

xandro0777 commented Nov 26, 2024 via email

@xandro0777
Copy link
Author

xandro0777 commented Nov 26, 2024 via email

@xandro0777
Copy link
Author

xandro0777 commented Nov 27, 2024 via email

@cketti
Copy link
Member

cketti commented Nov 27, 2024

More stack traces are unlikely to help. The stack trace we have shows that none of our code is directly involved in the crash. It's most likely a bug in Android that Thunderbird for Android is triggering. However, without knowing what triggers the bug we don't know how to work around it. Similarly, without knowing how to trigger the bug, Google will have a hard time fixing the bug in Android.

@xandro0777
Copy link
Author

xandro0777 commented Nov 27, 2024 via email

@xandro0777
Copy link
Author

xandro0777 commented Nov 27, 2024 via email

@xandro0777
Copy link
Author

xandro0777 commented Nov 27, 2024 via email

@cketti
Copy link
Member

cketti commented Dec 3, 2024

I'm finally able to reproduce this error. It depends on the length of the email address and the available display space.

Steps to reproduce:

  1. Open message compose screen while device is in landscape orientation.
  2. Enter an email address in the "To" field that takes up almost all of the available horizontal space.
  3. Make sure the entered email address is converted into a "chip", e.g. by focusing another input field.
  4. Rotate the device.

I haven't figured out yet what exactly is causing the crash. It might be a combination of us doing uncommon things to render the email address views and a bug in Android.

@cketti cketti removed the status: answered The issue was answered and is waiting for maintainer review. label Dec 3, 2024
@xandro0777
Copy link
Author

xandro0777 commented Dec 3, 2024 via email

@gitrickm
Copy link

I came to report this same bug. I'm running on a Galaxy S23 Ultra, Android 14. The same bug exists in K-9 Mail, but now I've upgraded to Thunderbird.

It fails on all compose actions: new mail, Reply All, edit a Draft. It either goes back to the Inbox, or crashes completely, when you rotate from portrait to landscape and back.

However, this only happens if your recipient list has more than 1 address. This is likely why the bug has existed for so long and doesn't seem to be repeatable. Now you have the missing piece of info! (Oh, I see someone says it also fails if a single recipient address is long.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something is causing incorrect behavior or errors type: crash Issues that involve fixing a crash or ANR
Projects
None yet
Development

No branches or pull requests

4 participants