-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[HOLD for 13349 & 55010] Re-enable the edge to edge mode on Android #55407
Comments
Triggered auto assignment to @Christinadobrzyn ( |
@kirillzyusko @chrispader @hannojg if you would like to take this on, please comment on the issue |
@mountiny, @Christinadobrzyn Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@chrispader is going to take this over 👀 |
@Kicu seems like this will be done with the hybrid app fragment PR |
Noting its these two PRs:
@war-in @Kicu Can you please comment here? We should monitor if all the paddings are right after the PR |
Hi, yes! I'm Marcin from Software Mansion and we think this issue should be resolved in mentioned PRs 🎉 |
1 similar comment
I'm going to move this to weekly while we wait on those PRs |
We have a specific case which is most likely also related to the edge to edge mode |
We just merged the fragments PR in the android hybrid app, I believe we might want to recheck where we are at now |
Just checking - @mountiny is rechecking something that we should ask QA to do or something we should ask the team working on this Gh to do? |
@war-in @staszekscp coul dyou please check if there is anything more we need to do now to re-enable the edge to edge? |
@mountiny I think the edge-to-edge mode is correctly enabled in HybridApp now and the last thing to do is to remove @staszekscp's PR disabling the mode in OD. Let's wait for @staszekscp to confirm and I'll prepare the revert in the meantime |
The revert is ready: https://github.com/Expensify/Mobile-Expensify/pull/13415 |
The fix was applied so the |
@staszekscp @war-in correct, the issue was in Android 15 specifically and the top was overlapping with the status bar which made it impossible to click on the oldDot buttons in the top. Can you please check if the oldDot works fine in Android 15 also after force closing the app etc |
It took some time but I finally have an answer and solution! I dug into the code and found places where improvements should be made. Finally, I managed to adjust the old app to handle edge-to-edge properly 🎉 (we no longer need to set paddings when switching between apps) Please, look at the videos and let me know what you think about the changes! Gesture navigation: Screen.Recording.Feb.10.2025.from.Expensify.Dev.mp4Button navigation: Screen.Recording.Feb.10.2025.from.Jan.and.Mateusz.mp4Sign-in page: Screen.Recording.Feb.10.2025.from.Marcin.Warchol.mp4 |
Problem
We had to disable the edge-to-edge mode in the hybrid app on Android 15 because it was causing critical UX issues. The edge-to-edge mode is standard and default in Android 15, and it provides better UX with immersive UI since both the top status bar and the bottom navigation bar connect with the rest of the App design smoothly.
Solution
Re-enable the edge to edge mode and confirm that the hybridApp on all common Android versions looks well.
@kirillzyusko @chrispader @staszekscp @WoLewicki Who would like to work on this? I assume Margelo would be good fit, they should get the hybriad app setup access soon too
The text was updated successfully, but these errors were encountered: