thunderbird-android icon indicating copy to clipboard operation
thunderbird-android copied to clipboard

The “Download Complete Message” button is located behind the bottom navigation bar and cannot be selected

Open vitacola45 opened this issue 6 months ago • 4 comments

Checklist

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

App

Thunderbird for Android

App version

11.0.b3

Where did you get the app from?

F-Droid

Android version

14 - One-UI 6.1

Device model

Samsung A52s 5G SM-A528B/DS

Steps to reproduce

  1. open a message
  2. scroll down

Expected behavior

the "Load complete message" button should appear above the lower navigation bar and be clickable

Actual behavior

Logs

No response

vitacola45 avatar Jun 17 '25 04:06 vitacola45

the same for me (Android 15, Moto G34)

michaelnum avatar Jun 17 '25 06:06 michaelnum

Thanks @vitacola45 and @michaelnum for the report! I also reproduced this using my Redmi A3, with Android 15, while using the 3-button navigation bar.

rafaeltonholo avatar Jun 17 '25 12:06 rafaeltonholo

Same for me (Android 15, OnePlus 12)

spymorass avatar Jun 18 '25 06:06 spymorass

The same for my Tablet (Samsing Tab A7 SM-T500; Android 12; One-UI 4.1

vitacola45 avatar Jun 19 '25 13:06 vitacola45

Also experiencing this

tarmacalastair avatar Jun 22 '25 22:06 tarmacalastair

I found the cause of this, and I was able to reproduce it on multiple devices here. I'll send a pull request with the solution.

The UI was being overlapped by system UI elements (like navigation bars and display cutouts) because WindowInsets were not properly applied to the root container.

Before After
Image Image

joohnq avatar Jun 23 '25 15:06 joohnq

I found the cause of this, and I was able to reproduce it on multiple devices here. I'll send a pull request with the solution.

The UI was being overlapped by system UI elements (like navigation bars and display cutouts) because WindowInsets were not properly applied to the root container.

Before:

After:

Hi @joohnq, unfortunately, as we want to have a fix on that issue before our next beta release, I went ahead and fixed the issue. If you are still interested in contributing to the project, I can try to find another good first issue for you to work on.

rafaeltonholo avatar Jun 23 '25 16:06 rafaeltonholo

Oh, @rafaeltonholo, I understand... I'll look for another issue. Sorry about the PR — I sent it before seeing your comment.

joohnq avatar Jun 23 '25 16:06 joohnq

Oh, @rafaeltonholo, I understand... I'll look for another issue. Sorry about the PR — I sent it before seeing your comment.

All good, don't need to apologize. Leave your PR open, I will close mine and we can proceed with your changes after reviewing

rafaeltonholo avatar Jun 23 '25 16:06 rafaeltonholo

Okay. :) I noticed you fixed the issue in a different file than mine. Could you explain why? I’m curious how your solution solves the problem differently.

Looking forward to your review.

joohnq avatar Jun 23 '25 16:06 joohnq

Okay. :) I noticed you fixed the issue in a different file than mine. Could you explain why? I’m curious how your solution solves the problem differently.

Looking forward to your review.

Sure, let's move this conversation to the pull request. I will review it shortly.

rafaeltonholo avatar Jun 23 '25 16:06 rafaeltonholo