Skip to content

fix: adapt screens to infinite edges feature - #4887

Merged
joragua merged 7 commits into
masterfrom
fix/infinite_edges
Jun 17, 2026
Merged

fix: adapt screens to infinite edges feature#4887
joragua merged 7 commits into
masterfrom
fix/infinite_edges

Conversation

@joragua

@joragua joragua commented Jun 11, 2026

Copy link
Copy Markdown
Member

Related Issues

App: #4576

  • Add changelog files for the fixed issues in folder changelog/unreleased. More info here
  • Add feature to Release Notes in ReleaseNotesViewModel.kt creating a new ReleaseNote() with String resources (if required)

QA

#4887 (comment)

Reports:

@joragua joragua self-assigned this Jun 11, 2026
@joragua joragua linked an issue Jun 11, 2026 that may be closed by this pull request
@joragua
joragua force-pushed the fix/infinite_edges branch 5 times, most recently from e00f447 to e43a125 Compare June 15, 2026 11:56
@joragua
joragua requested a review from jesmrec June 15, 2026 12:15
@joragua
joragua marked this pull request as ready for review June 15, 2026 12:15

@jesmrec jesmrec left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a couple of comments @joragua

@joragua
joragua requested a review from jesmrec June 16, 2026 07:56

@jesmrec jesmrec left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CR Approved, moving to QA

@jesmrec

jesmrec commented Jun 16, 2026

Copy link
Copy Markdown
Member

QA checks

Android 16 Android 15 Android 14 Reports
Settings
Passcode FIXED: BUG
Pattern FIXED: BUG
Log Files FIXED: BUG
Auto Uploads
Drawer ‼️ BUG
Sometimes a black band in landscape
Top Bar ‼️ ‼️  PARTIALLY FIXED BUG
Bottom bar WONT FIX: BUG
Folder Picker
Receive External file FIXED: BUG
Preview image FIXED: BUG
Preview video FIXED: BUG
FIXED: BUG
Preview audio FIXED: BUG
FIXED: BUG
Preview text FIXED: BUG
Share
Spaces
Uploads
Priv. policy
Release Notes

@jesmrec

jesmrec commented Jun 16, 2026

Copy link
Copy Markdown
Member

(1) [FIXED]

About passcode & pattern views:

  • Top bar not visible (white)
  • Titles are closer to the top bar (in Android 16's Pattern are overlapped)
  • Also reproducible in landscape
Android 15 Android 16
Screenshot_20260616_101906 share_7369714149060048114
Screenshot_20260616_102552 share_8174983675135640628

Xiaomi Redmi Note 13, Android 15
Pixel 7, Android 16
6fcc322f

@jesmrec

jesmrec commented Jun 16, 2026

Copy link
Copy Markdown
Member

(2) [FIXED]

About the Receive External File view:

Screenshot_20260616_103304

Xiaomi Redmi Note 13, Android 15
Pixel 7, Android 16
6fcc322f

@jesmrec

jesmrec commented Jun 16, 2026

Copy link
Copy Markdown
Member

(3) [FIXED]

Log file list

Screenshot_20260616_103927

Xiaomi Redmi Note 13, Android 15
Pixel 7, Android 16
6fcc322f

@jesmrec

jesmrec commented Jun 16, 2026

Copy link
Copy Markdown
Member

(4) [WONT FIX]

Check this difference when the device has control buttons in landscape. Not very important, but, everything could be improved:

Current v4.7.0
Screenshot_20260616_105433 Screenshot_20260616_105306

Xiaomi Redmi Note 13, Android 15
6fcc322f

@jesmrec

jesmrec commented Jun 16, 2026

Copy link
Copy Markdown
Member

(5) [FIXED]

About the drawer in landscape orientation:

Before: list of menu options and quota were scrolled at the time
Now: list of menu options and quota can collide during scroll:

Current v4.7.0
https://github.com/user-attachments/assets/66098cac-22ec-4d69-90e2-5d8f4bbf1b9e https://github.com/user-attachments/assets/41e054df-944e-485e-b086-5d1bb6de7b88

Xiaomi Redmi Note 13, Android 15
6fcc322f

joragua added 3 commits June 16, 2026 13:25
Signed-off-by: Jorge Aguado Recio <jorgeagurec@gmail.com>
Signed-off-by: Jorge Aguado Recio <jorgeagurec@gmail.com>
Signed-off-by: Jorge Aguado Recio <jorgeagurec@gmail.com>
@joragua
joragua force-pushed the fix/infinite_edges branch from 6fcc322 to cd96a1e Compare June 16, 2026 11:26
@joragua

joragua commented Jun 16, 2026

Copy link
Copy Markdown
Member Author

(1), (2) and (3) should be fixed now

Regarding the passcode and pattern screens, I’ve added the toolbar to the XML and used the adaptInfiniteEdges() extension. The others screens were easier -> use the extension

@jesmrec

jesmrec commented Jun 16, 2026

Copy link
Copy Markdown
Member

(1), (2) and (3) fixed

@jesmrec

jesmrec commented Jun 16, 2026

Copy link
Copy Markdown
Member

(6) [FIXED]

In Android 15 or 16, preview an image or a video

Current: Toolbar is not visible
Expected:: Toolbar visible

Current v4.7.0
share_7622123632016283693 Screenshot 2026-06-16 at 14 24 56

Xiaomi Redmi Note 13, Android 15
Pixel 7, Android 16
cd96a1e33

@jesmrec

jesmrec commented Jun 16, 2026

Copy link
Copy Markdown
Member

(7) [FIXED]

The multiselection operations menu's background in now black instead of white.

Screenshot 2026-06-16 at 14 15 48

Xiaomi Redmi Note 13, Android 15
Pixel 7, Android 16
cd96a1e33

@jesmrec

jesmrec commented Jun 16, 2026

Copy link
Copy Markdown
Member

(8) [FIXED]

In the video/audio player, the bottom bar collides with the control buttons

Screenshot 2026-06-16 at 14 32 08

Xiaomi Redmi Note 13, Android 15
cd96a1e33

joragua added 2 commits June 16, 2026 15:46
Signed-off-by: Jorge Aguado Recio <jorgeagurec@gmail.com>
Signed-off-by: Jorge Aguado Recio <jorgeagurec@gmail.com>
@joragua
joragua force-pushed the fix/infinite_edges branch from 29ba9ed to 8dfe9e7 Compare June 16, 2026 13:49
@joragua

joragua commented Jun 16, 2026

Copy link
Copy Markdown
Member Author

(6) and (7) should be fixed in commit 353237f -> Removed theme in PasscodeActivity and PatternActivity from manifest and restore the theme in the default action bar

(8) should be fixed in commit 8dfe9e7 -> Adapt content to infinite edges feature (the same as the other screens)

@jesmrec

jesmrec commented Jun 16, 2026

Copy link
Copy Markdown
Member

(6), (7) and (8) fixed

@jesmrec

jesmrec commented Jun 16, 2026

Copy link
Copy Markdown
Member

(9) [PARTIALLY FIXED]

Setting the mobile in landscape and enabling multiselection mode, top bar looks like:

Screenshot 2026-06-16 at 17 27 13

It does not reach the screen edges and shows the stuff behind

Xiaomi Redmi Note 13, Android 15
Pixel 7, Android 16
8dfe9e720

@joragua

joragua commented Jun 16, 2026

Copy link
Copy Markdown
Member Author

Regarding (4): the sort menu is inside the fragment content, so it cannot be adapted to the edge-to-edge in a different way. It may look there is a strange white rectangle, but that's because the sort menu has the same color as the toolbar. There is one solution but it has an issue: the app buttons cannot be clicked due to the navigation system bar, so I would not apply that fix. I'd mark the report as a wontfix, but I'll open a new issue and wait to see if someone has a different solution 🤔

Signed-off-by: Jorge Aguado Recio <jorgeagurec@gmail.com>
@joragua

joragua commented Jun 17, 2026

Copy link
Copy Markdown
Member Author

Regarding (9), I’ve hidden the toolbar when multi-selection mode is enabled in order to avoid showing the avatar and the hamburger menu in landscape (fixed in this commit: 68d9552 ). Anyway I'll create a new issue to try to avoid glitches with the edge-to-edge feature (only reproducible on smartphones in landscape mode) because probably it's necessary to create a new design for the action bar.

…d user quota

Signed-off-by: Jorge Aguado Recio <jorgeagurec@gmail.com>
@joragua
joragua force-pushed the fix/infinite_edges branch from 66bebfa to 66dcf89 Compare June 17, 2026 12:24
@joragua

joragua commented Jun 17, 2026

Copy link
Copy Markdown
Member Author

(5) should be fixed now in commit 66dcf89

Restored the drawer structure and applied insets to the first NavigationView in the layout

@jesmrec

jesmrec commented Jun 17, 2026

Copy link
Copy Markdown
Member

Approved from my side

Reports (4) and (9) will be addressed to separate issues because the fix will require deep changes that go over the scope of the current issue.

@joragua
joragua merged commit 13ae7c8 into master Jun 17, 2026
10 of 11 checks passed
@joragua
joragua deleted the fix/infinite_edges branch June 17, 2026 16:27
@DeepDiver1975 DeepDiver1975 mentioned this pull request Jun 26, 2026
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Infinite edges (starting from Android 15)

2 participants