[$250] IOS - Empty screen seen when navigating back from workspace description the second time

[lanitochka17]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Version Number: 9.0.71-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): nathanmulugetatesting+2338@gmail.com
Issue reported by: Applause - Internal Team

Action Performed:

1. Open IOS hybrid app
2. Create a workspace give it a custom description
3. Navigate to the inbox
4. Open workspace chat
5. Tap on the custom description to navigate to description setting of the workspace
6. Navigate back twice by tapping on the top left back icon until you reach workspace editor page
7. Tap on Workspace chat
8. Re do step 5
9. Tap on the top corner back icon or swipe from left to right

Expected Result:

Empty screen does not get shown

Actual Result:

Empty screen gets shown and the workspace profile settings page animation is wrong, it slides from right to left after the empty screen has been shown first

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

• Android: Standalone
• Android: HybridApp
• Android: mWeb Chrome
• iOS: Standalone
• iOS: HybridApp
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6684551_1733335793079.ScreenRecording_12-04-2024_21-02-36_1.1.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021865096772528155887
• Upwork Job ID: 1865096772528155887
• Last Price Increase: 2024-12-20

Issue Owner

Current Issue Owner: @ishpaul777

[melvin-bot]

Triggered auto assignment to @muttmuure (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.

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~021865096772528155887

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @ishpaul777 (External)

[ikevin127]

Proposal

Please re-state the problem that we are trying to solve in this issue

Empty screen gets shown and the workspace profile settings page animation is wrong, it slides from right to left after the empty screen has been shown first.

What is the root cause of that problem?

TLDR: The same RCA detailed in my proposal of this other issue.

Note

This issue is only present on narrow layout devices (native / mWeb) where screens displayed in RHP act as a modal.

App/src/pages/workspace/WorkspaceInitialPage.tsx

Lines 360 to 376 in 3baf965

	// We are checking if the user can access the route.
	// If user can't access the route, we are dismissing any modals that are open when the NotFound view is shown
	const canAccessRoute = activeRoute && menuItems.some((item) => item.routeName === activeRoute);
	useEffect(() => {
	if (!shouldShowNotFoundPage && canAccessRoute) {
	return;
	}
	if (wasRendered.current) {
	return;
	}
	wasRendered.current = true;
	// We are dismissing any modals that are open when the NotFound view is shown
	Navigation.isNavigationReady().then(() => {
	Navigation.closeRHPFlow();
	});
	}, [canAccessRoute, shouldShowNotFoundPage]);

When we open the workspace's initial page, the active route becomes Workspace_Initial. However, the current canAccessRoute logic only checks for the sub-page route - therefore when it compares the active route with itself, it fails because menuItems doesn't contain Workspace_Initial.

We actually have the logic to only run it once here, but when we open the search page, all routes are "hidden" as explained here. Because of this, the wasRendered logic in WorkspaceInitialPage won't work as expected -> when we try to open the filter page, the previously "hidden" routes are added back to the stack and canAccessRoute becomes false which will dismiss any RHP that we try to open (Filters).

Credits to bernhardoj for the RCA.

What changes do you think we should make in order to solve the problem?

We had a similar issue which I reviewed recently here where we navigated away from WorkspaceProfileSharePage to a report without dismissing the currently opened modal which caused a weird UI bug in the QR Code because the same modal would be opened twice.

Same as proposed in the other issue, a practical solution for this kind of problem is to simply replace the way we navigate away from WS to the report here:

App/src/pages/workspace/WorkspaceInitialPage.tsx

Line 457 in b595f5d

onPress={() => Navigation.navigate(ROUTES.REPORT_WITH_ID.getRoute(currentUserPolicyExpenseChat?.reportID ?? '-1'))}

to use Navigation.dismissModal instead:

onPress={() => Navigation.dismissModal(currentUserPolicyExpenseChat?.reportID ?? '-1')}

Why dismissing the modal first fixes our issue ?
Because as explained in RCA, on narrow layout devices (native / mWeb) screens displayed in RHP act as a modal and by dismissing it when navigating to the report, all the conditions which lead to the empty screen showing when navigating back in our case, will now work as expected.

Important

I proposed the same solution for this issue and because this solution would fix both issues, if my solution is selected on any of the two issues -> we should put the other one on HOLD.

Result video (before / after)

iOS: Native

Before	After
before.mp4	after.mp4

[melvin-bot]

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

[ishpaul777]

i'll try to priortize the review today 🙇

[ishpaul777]

@ikevin127 Will this issue be fixed in #53600, i see we got a proposal selected there?

[ikevin127]

Will this issue be fixed in #53600, i see we got a proposal selected there?

cc @bernhardoj for input on this question

[bernhardoj]

No

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[ishpaul777]

will review over weekend 🙏

[ishpaul777]

I tried your solution @ikevin127 It does solve the blank screen issue but when navigating back from report it goes to workspace list page instead of workspace settings since we dismissed Mosal. while its indeed better than the current behaviour i dont think its the ideal behaviour

Screen.Recording.2024-12-17.at.12.15.02.AM.mov

[ikevin127]

Expected result states:

Empty screen does not get shown

From you video it looks like that the expected result is fulfilled with the proposed solution since the expected result did not specify what page we should land on. But ok, I'll look into this and try to come up with an alternative solution that would fulfill your expectations 👍

[melvin-bot]

@muttmuure @ishpaul777 this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

[ishpaul777]

awaiting proposals

[ikevin127]

I performed a npm ci and iOS: Native app rebuild and I can't reproduce the issue anymore.

ios.mp4

[bernhardoj]

I think it was "solved" by #53985, but it causes the navigation issue mentioned here : #54303.

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[ishpaul777]

i am no longer able to reproduce the but having workspace profile page in background feels odd,

Screen.Recording.2024-12-24.at.12.07.38.AM.mov

@Expensify/design @muttmuure any thoughts on this? i think the desired behaviour is that if press back '<' from description user should land on report page not to workspace profile page..

[shawnborton]

I think the video looks correct? It seems to be navigating sequentially based on the browsing history.

[ishpaul777]

I think the video looks correct? It seems to be navigating sequentially based on the browsing history.

it seems it first navigates to workspace profile page then navigate to description which seems little odd to me, is the extra screen in between necessary?

if that seems ok i think we can close this one, since issue in OP not reproducable anymore