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

[HOLD for 13349 & 55010] Re-enable the edge to edge mode on Android #55407

Open
mountiny opened this issue Jan 17, 2025 · 19 comments
Open

[HOLD for 13349 & 55010] Re-enable the edge to edge mode on Android #55407

mountiny opened this issue Jan 17, 2025 · 19 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Reviewing Has a PR in review Weekly KSv2

Comments

@mountiny
Copy link
Contributor

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

@mountiny mountiny added Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 labels Jan 17, 2025
Copy link

melvin-bot bot commented Jan 17, 2025

Triggered auto assignment to @Christinadobrzyn (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@mountiny
Copy link
Contributor Author

@kirillzyusko @chrispader @hannojg if you would like to take this on, please comment on the issue

@mountiny mountiny self-assigned this Jan 17, 2025
@melvin-bot melvin-bot bot added the Overdue label Jan 20, 2025
Copy link

melvin-bot bot commented Jan 21, 2025

@mountiny, @Christinadobrzyn Whoops! This issue is 2 days overdue. Let's get this updated quick!

@kirillzyusko
Copy link
Contributor

@chrispader is going to take this over 👀

@mountiny
Copy link
Contributor Author

@Kicu seems like this will be done with the hybrid app fragment PR

@melvin-bot melvin-bot bot removed the Overdue label Jan 21, 2025
@mountiny
Copy link
Contributor Author

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

@war-in
Copy link
Contributor

war-in commented Jan 21, 2025

Hi, yes! I'm Marcin from Software Mansion and we think this issue should be resolved in mentioned PRs 🎉

@mountiny mountiny added the Reviewing Has a PR in review label Jan 21, 2025
@Christinadobrzyn Christinadobrzyn changed the title Re-enable the edge to edge mode on Android [HOLD for 13349 & 55010] Re-enable the edge to edge mode on Android Jan 21, 2025
@Christinadobrzyn
Copy link
Contributor

1 similar comment
@Christinadobrzyn
Copy link
Contributor

@Christinadobrzyn
Copy link
Contributor

I'm going to move this to weekly while we wait on those PRs

@ZhenjaHorbach
Copy link
Contributor

ZhenjaHorbach commented Feb 3, 2025

@war-in

We have a specific case which is most likely also related to the edge to edge mode

#56205 (comment)

@mountiny
Copy link
Contributor Author

mountiny commented Feb 4, 2025

We just merged the fragments PR in the android hybrid app, I believe we might want to recheck where we are at now

@Christinadobrzyn
Copy link
Contributor

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?

@mountiny
Copy link
Contributor Author

mountiny commented Feb 6, 2025

@war-in @staszekscp coul dyou please check if there is anything more we need to do now to re-enable the edge to edge?

@war-in
Copy link
Contributor

war-in commented Feb 6, 2025

@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.
Currently, we enable it programmatically, so the line in themes.xml is just ignored

Let's wait for @staszekscp to confirm and I'll prepare the revert in the meantime

@war-in
Copy link
Contributor

war-in commented Feb 6, 2025

@staszekscp
Copy link
Contributor

The fix was applied so the edge-to-edge doesn't break the OldDot part of the app - as far as I remember the problem was that users were unable to press buttons on the top of the screen on Android 15 - did you have chance to see if everything works fine there?

@mountiny
Copy link
Contributor Author

mountiny commented Feb 6, 2025

@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

@war-in
Copy link
Contributor

war-in commented Feb 10, 2025

It took some time but I finally have an answer and solution!
@staszekscp and @mountiny you were right, after removing the flag, the status and nav bars were broken on android 15. The color was always white, even on the sign-in page.

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.mp4

Button navigation:

Screen.Recording.Feb.10.2025.from.Jan.and.Mateusz.mp4

Sign-in page:

Screen.Recording.Feb.10.2025.from.Marcin.Warchol.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Reviewing Has a PR in review Weekly KSv2
Projects
Status: No status
Development

No branches or pull requests

6 participants